Skip to content

Fix flake8 to fix Travis CI #705

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Fix flake8 to fix Travis CI #705

wants to merge 1 commit into from

Conversation

hugovk
Copy link
Contributor

@hugovk hugovk commented Dec 4, 2017

No description provided.

@hugovk hugovk mentioned this pull request Dec 4, 2017
Closed
@codecov-io
Copy link

Codecov Report

Merging #705 into master will increase coverage by 0.99%.
The diff coverage is 89.74%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #705      +/-   ##
==========================================
+ Coverage   92.83%   93.83%   +0.99%     
==========================================
  Files          61       60       -1     
  Lines       10152     9343     -809     
==========================================
- Hits         9425     8767     -658     
+ Misses        727      576     -151
Impacted Files Coverage Δ
git/test/test_submodule.py 97.52% <ø> (ø) ⬆️
git/repo/base.py 95.29% <0%> (ø) ⬆️
git/refs/log.py 93.84% <0%> (ø) ⬆️
git/test/lib/helper.py 87.79% <0%> (ø) ⬆️
git/test/test_commit.py 100% <100%> (ø) ⬆️
git/compat.py 64.81% <100%> (ø) ⬆️
git/exc.py 96.42% <100%> (ø) ⬆️
git/diff.py 99.11% <100%> (ø) ⬆️
git/test/test_util.py 88.23% <100%> (ø) ⬆️
... and 3 more

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 0a96030...c467cc0. Read the comment docs.

@hugovk
Copy link
Contributor Author

hugovk commented Dec 5, 2017

Duplicate of #702.

@hugovk hugovk changed the title Fix flake8 to fix CI Fix flake8 to fix Travis CI Dec 5, 2017
cwonrails added a commit to cwonrails/GitPython that referenced this pull request Dec 7, 2017
@cwonrails
Merge upstream gitpython-developers#705 & gitpython-developers#706 PRs
936a03d 
* github.com:gitpython-developers/GitPython:
  Remove redundant Python 2.4 code
  Specify Python 3.6 support
  Fix flake8
@Byron
Copy link
Member

Byron commented Dec 11, 2017

Thanks for the fix! As stated in the previous PR, I merged the fixes earlier already.

@Byron Byron closed this Dec 11, 2017
@hugovk hugovk deleted the fix-ci branch December 11, 2017 14:50
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@hugovk @codecov-io @Byron