Skip to content

Scrabble #4

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
Open
wants to merge 33 commits into
base: master
Choose a base branch
from
Open

Scrabble #4

wants to merge 33 commits into from

Conversation

BlueColorPencils
Copy link

Leah and Cristal worked on this.
No additional things needed.

leahnp and others added 30 commits March 7, 2016 16:50
@leahnp
trying to commit entire directory
f7c64f0
Created spec/lib folder and respective files
2e5a8ea
@leahnp
Added Rakefile and first test, made first test pass
6d22def
Required simplecov in spec helper. Required player and scoring into s…
1102970 
…crabble.rb.
Last commit was not up to date.
507e44d
@leahnp
Added a hash to store values and letters, we still don't think the fi…
44ecb0c 
…les are set up right..
@leahnp
required ./scrabble in the scoring and player rb files
7576934
trying to get rid of require error messages with jeremy. this sucks
9a5aa15
changed capital S to lowercase.rb
e031d9a
fixed require errors by taking out require scrabble in scoring.rb. Ma…
1739dc6 
…de .test method return MEOW. Figured out how to effectively run all test files and methods.
@leahnp
got self.score test passing, method is generating score for single word
38ff41a
@leahnp
created method and test for highest_score_from which takes in array, …
cf4f49c 
…creates scores for each word in array and returns the highest scored word
reworked highest_score_from to store score and words in separate arra…
4db0688 
…ys, then zip them and sort. And dropped all arrays that were not equal to the highest score.
@leahnp
Added conditional to deal with same length words, returns word which …
9987926 
…appeared first in original array of words as winner
Accounted for 7 letter words in both scoring methods. Commented code. A
a4826c6
Added more comments. Looked at index.html and deleted useless lines o…
9240c59 
…f code. Added a ZEDREE test. Everything rakes.
@leahnp
Created a method to clean up redudency in searching arrays for ties
81a8b84
added basic wave2 methods to player and started a player_spec.
3ac492e
@leahnp
Added played_words in initializer, made it an attr_reader and created…
5bde066 
… a test to test that it returns an array of played words
@leahnp
Added play(words) method and test to take in newly played word and re…
66d9546 
…turn score and add to played_words array
Added total_score method and test. It works..
39c677c
Added won? method and called it into play(word). Next we will work on…
f1711d7 
… highest_scoring_word.
@leahnp
Added highest_scoring_word and highest_word_score methods and tests
9667c20
added the entire tilebag class and specs without interim commits.. wo…
62f6cc6 
…ops. Everything passes though.
messed up last push (twas a mere folder). this should have everything.
0bf15af
@leahnp
Start by correcting tiles_method to just remove played chars and repo…
08a7ed0 
…pulated tiles
To fix player.rb we had to create an instance of tilebag and refer to…
15b51f7 
… it to call methods. Added draw method. Everything passes.
@leahnp
Cleaned up player_spec, add two final tests with two player instances.
1306474
@leahnp
Cleaned up scoring_spec, added spec to check constant was set correctly
ac55e6b
@leahnp
Changed language in second scoring_spec spec
8122935
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.

2 participants

@BlueColorPencils @leahnp