Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Beylul & Danielle #14

Open
wants to merge 24 commits into
base: master
Choose a base branch
from
Open

Beylul & Danielle #14

wants to merge 24 commits into from

Conversation

dschrimm
Copy link

@dschrimm dschrimm commented Sep 2, 2016

Finished all requirements and added some option enhancements to Players class. Writing tests was difficult but proved useful to check whether seemingly small changes impacted other classes or methods.

dschrimm and others added 24 commits August 29, 2016 16:38
@dschrimm
Created initial module/class and data structure and added specs files.
31516d3
@Bella8
added the Rakefile, added one red test
d7141db
@dschrimm
Added a spec and score method appears to accurately return total score.
0156214
@Bella8
Added an ArgummentError and updated the test
11ff60f
@dschrimm
Added 7 letter bonus feature and completed super baseline highest_sco…
fde8a5d 
…re_from method.
@Bella8
Modified the highst score method
2f679de
@dschrimm
Added tie logic to highest_score_from
134f680
@Bella8
Added more tests that check the correct input for method highest_scor…
d63e9bc 
…e_from.
@dschrimm
started player and player specs. initialize, plays, and play methods.
b5fa027
@Bella8
Added the total_score method and tests
603bc0d
@dschrimm
Add won and associated methods and specs.
abbde38
@Bella8
Modified the specs/spec_helper.rb
eb5c285
@Bella8
Added highst_scoring_word and highst_word_score methods and tests
82bdb29
@dschrimm
Working version of Wave 2
b86a1ce
@dschrimm
Removed unnecessary plays method.
5130558
@dschrimm
added gitignore
1a6aba3
@Bella8
Created the tilebag file and the specs tilebag file. Added one test a…
cbe3889 
…nd created one method frame.
@dschrimm
Add draw-tiles method and change list of available tiles to array
acb407d
@Bella8
Added draw_tiles(tile_bag) method and more tests
5585516
@dschrimm
Additional player methods tiles and draw_tiles and associated specs
7c8b9ae
@Bella8
Added more tests and fixed failing three failing tests.
041a4da
@dschrimm
Accounted for checking tiles for double letter words
5d5f001
@dschrimm
Shorten letter initializer processs
0d3d760
@Bella8
Cleaned up the programs
cbe1f28
@kariabancroft
Copy link

HI HOW ARE YOU DOING @dschrimm @Bella8

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dschrimm @kariabancroft @Bella8