Skip to content

Conversation

@KatieButler
Copy link

This change is Review on Reviewable

@LucyWilcox
Copy link

Review status: 0 of 8 files reviewed at latest revision, 4 unresolved discussions.


frequency.py, line 27 [r1] (raw file):
This could all be done in a separate file that you only run once. Or it could be in a function and you could just comment out the function call instead just commenting it all out.e


frequency.py, line 50 [r1] (raw file):
Don't push commented out stuff you're not using.


frequency.py, line 84 [r1] (raw file):
Delete old stuff like this.


frequency.py, line 89 [r1] (raw file):
Would be nice if this was 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