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.

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

Staging #9

Merged
merged 3 commits into from
Feb 4, 2016
Merged

Staging #9

merged 3 commits into from
Feb 4, 2016

Conversation

genome21
Copy link
Owner

@genome21 genome21 commented Feb 4, 2016

Edits to code style to remove common errors

Review on Reviewable

…ausing issues when handling the updates.txt file.

I have implemented the WITH class to ensure that the file is properly and safely opened and closed.
@landscape-bot
Copy link

Code Health
Repository health decreased by 40% when pulling 40e651a on staging into 1b03c81 on master.

@landscape-bot
Copy link

Code Health
Code quality remained the same when pulling c425acb on genome21:staging into 1b03c81 on genome21:master.

@genome21
Copy link
Owner Author

genome21 commented Feb 4, 2016

Reviewed 1 of 1 files at r2.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from the review on Reviewable.io

genome21 added a commit that referenced this pull request Feb 4, 2016
@genome21 genome21 merged commit 2137fa8 into master Feb 4, 2016
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