Skip to content
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

Configure bdist_wheel in travis fixes #58 #127

Merged
merged 1 commit into from
Feb 22, 2017

Conversation

graingert
Copy link
Contributor

No description provided.

@graingert
Copy link
Contributor Author

@jettify you should configure a different user with maintainer only access rights to aiobotocore, rather than using your own password in your travis config.

@codecov-io
Copy link

codecov-io commented Feb 22, 2017

Codecov Report

Merging #127 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #127   +/-   ##
=======================================
  Coverage   81.63%   81.63%           
=======================================
  Files           7        7           
  Lines         392      392           
=======================================
  Hits          320      320           
  Misses         72       72

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 3ca3aae...7a122d0. Read the comment docs.

@jettify jettify merged commit 0adb2c8 into aio-libs:master Feb 22, 2017
@jettify
Copy link
Member

jettify commented Feb 22, 2017

github/travis provides capability to do deployment, without any need to touch pypi, do not see any reason to do deployment by hand.

@graingert
Copy link
Contributor Author

Sure but Travis only needs maintainership rights to one pypi project. No need to give it your account with ownership access to many projects

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

Successfully merging this pull request may close these issues.

3 participants