Skip to content

Finished Word Frequency Toolbox#43

Open
laurengulland wants to merge 1 commit intosd16spring:masterfrom
laurengulland:master
Open

Finished Word Frequency Toolbox#43
laurengulland wants to merge 1 commit intosd16spring:masterfrom
laurengulland:master

Conversation

@laurengulland
Copy link

This change is Review on Reviewable

@LucyWilcox
Copy link

Review status: 0 of 3 files reviewed at latest revision, 2 unresolved discussions.


frequency.py, line 54 [r1] (raw file):
You could just splice wordList.


frequency.py, line 67 [r1] (raw file):
S and t aren't good variable names and this could stand to be in an if name equals main statement.


Comments from the review on Reviewable.io

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants