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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Made the code PEP8-compliant #1476

Merged
merged 104 commits into from Jan 30, 2015
Merged

Made the code PEP8-compliant #1476

merged 104 commits into from Jan 30, 2015

Conversation

@aviau
Copy link
Contributor

@aviau aviau commented Jan 27, 2015

So yeah. We did some PEP8 tonight. All tests pass. We have also added a PEP8 check to Travis.

//cc @Seb-Solon @gst @titilambert @Freddrickk 馃嵒 馃嵒

REBASES ARE COMING

@aviau aviau changed the title Pylint party Made the code PEP8-compliant Jan 27, 2015
@coveralls
Copy link

@coveralls coveralls commented Jan 27, 2015

Coverage Status

Coverage remained the same at % when pulling b7133f7 on savoirfairelinux:PylintParty into 025964a on naparuba:master.

@naparuba
Copy link
Owner

@naparuba naparuba commented Jan 30, 2015

Merge in progress, just a conflict and tests are running

@naparuba naparuba merged commit b7133f7 into naparuba:master Jan 30, 2015
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
@naparuba
Copy link
Owner

@naparuba naparuba commented Jan 30, 2015

GG all ^^

@gst gst deleted the savoirfairelinux:PylintParty branch Jan 30, 2015
@aviau

This comment has been minimized.

Copy link
Contributor

@aviau aviau commented on shinken/webui/bottlewebui.py in 65e5d08 Jan 30, 2015

@Freddrickk oops!

I opened a PR to fix: #1487

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants