Skip to content

Fix failing Travis build due to issues 39-45 #46

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

Merged
merged 4 commits into from
Sep 28, 2020

Conversation

vduseev
Copy link
Contributor

@vduseev vduseev commented Sep 26, 2020

Summary

Fixes several issues that prevented successful Travis build.

Affected issues

Fixes #39
Fixes #41
Fixes #43
Fixes #45

Build results

Link to successful Travis build: https://travis-ci.org/github/vduseev/python-configuration/builds/730480955

Thoughts

Maybe we can add couple new settings to existing Travis configuration?

  • Build project periodically, once a month/week just to check that everything is working;
  • Rerun failed jobs 1-3 times because sometimes PyPi might suddenly interrupt the connection resulting in build failure.

Poetry is now installed using official `get-poetry.py` script using --preview
flag.
Script is downloaded to /tmp directory and poetry environment is sourced
into the Travis' shell.
@codecov-commenter
Copy link

codecov-commenter commented Sep 26, 2020

Codecov Report

Merging #46 into master will decrease coverage by 0.39%.
The diff coverage is 98.57%.

Impacted file tree graph

@@             Coverage Diff             @@
##            master      #46      +/-   ##
===========================================
- Coverage   100.00%   99.60%   -0.40%     
===========================================
  Files            6        7       +1     
  Lines          597      760     +163     
  Branches       116      150      +34     
===========================================
+ Hits           597      757     +160     
- Misses           0        1       +1     
- Partials         0        2       +2     
Impacted Files Coverage Δ
config/configuration.py 98.13% <91.66%> (-1.87%) ⬇️
config/__init__.py 100.00% <100.00%> (ø)
config/configuration_set.py 100.00% <100.00%> (ø)
config/contrib/aws.py 100.00% <100.00%> (ø)
config/contrib/azure.py 100.00% <100.00%> (ø)
config/contrib/gcp.py 100.00% <100.00%> (ø)
config/helpers.py 100.00% <100.00%> (ø)

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 c1913de...98dd9bd. Read the comment docs.

@vduseev
Copy link
Contributor Author

vduseev commented Sep 26, 2020

I don't have that much experience with using codecov. Could somebody please help with fixing this 0.4% not covered due to the fix added for #45?

@tr11 tr11 merged commit 3828ff6 into tr11:master Sep 28, 2020
@vduseev
Copy link
Contributor Author

vduseev commented Sep 28, 2020

Thank you @tr11 for merging the PR and adding all the required annotations for codecov to make it pass. Such a swift review and consideration from your side!

P.S. python-configuration is beautiful ❤️ I absolutely love it

@tr11
Copy link
Owner

tr11 commented Sep 28, 2020

You're very welcome. Always looking for people to contribute to the project 👍

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