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

Fixes #113 - vendor-in git requirements #120

Merged
merged 8 commits into from Oct 12, 2017

Conversation

Projects
None yet
2 participants
@jantman
Owner

jantman commented Oct 12, 2017

This fixes #113 by vendoring-in the git requirements so now biweeklybudget can be fully installed via one pip command. This also updates the docs, and adds some tests for the command line entrypoints to the Docker build process.

@jantman jantman added the in progress label Oct 12, 2017

@codecov-io

This comment has been minimized.

Show comment
Hide comment
@codecov-io

codecov-io Oct 12, 2017

Codecov Report

Merging #120 into master will increase coverage by 4.18%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
+ Coverage   25.13%   29.32%   +4.18%     
==========================================
  Files          59       54       -5     
  Lines        4165     3570     -595     
  Branches      640      561      -79     
==========================================
  Hits         1047     1047              
+ Misses       3096     2502     -594     
+ Partials       22       21       -1
Impacted Files Coverage Δ
biweeklybudget/vendored/ofxclient/version.py 100% <ø> (ø)
biweeklybudget/ofxupdater.py 0% <0%> (ø) ⬆️
biweeklybudget/ofxgetter.py 0% <0%> (ø) ⬆️
biweeklybudget/backfill_ofx.py 0% <0%> (ø) ⬆️
biweeklybudget/version.py
biweeklybudget/vendored/wishlist/browser.py
biweeklybudget/vendored/wishlist/__init__.py
biweeklybudget/vendored/wishlist/__main__.py
biweeklybudget/vendored/wishlist/core.py
biweeklybudget/vendored/wishlist/compat.py
... and 1 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 9d0188d...f4004dd. Read the comment docs.

codecov-io commented Oct 12, 2017

Codecov Report

Merging #120 into master will increase coverage by 4.18%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
+ Coverage   25.13%   29.32%   +4.18%     
==========================================
  Files          59       54       -5     
  Lines        4165     3570     -595     
  Branches      640      561      -79     
==========================================
  Hits         1047     1047              
+ Misses       3096     2502     -594     
+ Partials       22       21       -1
Impacted Files Coverage Δ
biweeklybudget/vendored/ofxclient/version.py 100% <ø> (ø)
biweeklybudget/ofxupdater.py 0% <0%> (ø) ⬆️
biweeklybudget/ofxgetter.py 0% <0%> (ø) ⬆️
biweeklybudget/backfill_ofx.py 0% <0%> (ø) ⬆️
biweeklybudget/version.py
biweeklybudget/vendored/wishlist/browser.py
biweeklybudget/vendored/wishlist/__init__.py
biweeklybudget/vendored/wishlist/__main__.py
biweeklybudget/vendored/wishlist/core.py
biweeklybudget/vendored/wishlist/compat.py
... and 1 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 9d0188d...f4004dd. Read the comment docs.

@jantman jantman merged commit aa3ebc1 into master Oct 12, 2017

3 of 4 checks passed

codecov/patch 0% of diff hit (target 25.13%)
Details
codecov/project 29.32% (+4.18%) compared to 9d0188d
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jantman jantman deleted the issues/113 branch Oct 12, 2017

@jantman jantman removed the in progress label Oct 12, 2017

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