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

PEP-8 nearly all of fedmsg #422

Merged
merged 1 commit into from Jun 18, 2017

Conversation

Projects
None yet
3 participants
@jeremycline
Member

jeremycline commented Jun 16, 2017

This PEP-8s nearly all of fedmsg except for the stuff touched by #421.

Signed-off-by: Jeremy Cline jeremy@jcline.org

PEP-8 nearly all of fedmsg
This PEP-8s nearly all of fedmsg except for the stuff touched by #421.

Signed-off-by: Jeremy Cline <jeremy@jcline.org>

@jeremycline jeremycline force-pushed the jeremycline:pep8-the-world branch from 3cb2871 to 7fe5111 Jun 16, 2017

@codecov-io

This comment has been minimized.

codecov-io commented Jun 16, 2017

Codecov Report

Merging #422 into develop will not change coverage.
The diff coverage is 88.88%.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #422   +/-   ##
========================================
  Coverage    47.15%   47.15%           
========================================
  Files           33       33           
  Lines         2059     2059           
  Branches       331      331           
========================================
  Hits           971      971           
  Misses         993      993           
  Partials        95       95
Impacted Files Coverage Δ
fedmsg/consumers/relay.py 100% <ø> (ø) ⬆️
fedmsg/encoding/__init__.py 50% <0%> (-1.52%) ⬇️
fedmsg/meta/__init__.py 71.53% <100%> (ø) ⬆️
fedmsg/replay/sqlstore.py 80.3% <100%> (ø) ⬆️
fedmsg/crypto/gpg.py 53.26% <100%> (+0.51%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0553d2...7fe5111. Read the comment docs.

@jeremycline jeremycline merged commit 5aad9af into fedora-infra:develop Jun 18, 2017

3 checks passed

codecov/patch 88.88% of diff hit (target 47.15%)
Details
codecov/project 47.15% (+0%) compared to f0553d2
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@jeremycline jeremycline deleted the jeremycline:pep8-the-world branch Jun 18, 2017

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