-
Notifications
You must be signed in to change notification settings - Fork 101
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
Initial travis config for pypi deployment #102
Conversation
Hello @georgeyk, |
Codecov Report
@@ Coverage Diff @@
## master #102 +/- ##
==========================================
+ Coverage 91.61% 92.37% +0.76%
==========================================
Files 6 6
Lines 298 354 +56
==========================================
+ Hits 273 327 +54
- Misses 25 27 +2
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems straightforward enough. I am not super familiar with the PyPi setup, do we need to add credentials in here as secure strings? @thinkingserious do you know?
Hello @georgeyk, |
Hello @georgeyk, |
Fixes
#92
Short description of what this PR does:
Adds the initial changes in travis configuration to integrate pypi as discussed in #92
@thinkingserious I'm not sure how I could test this change.