Skip to content

Conversation

@bittner
Copy link
Member

@bittner bittner commented Sep 30, 2018

@cool-RR cool-RR merged commit 5524e4d into PythonTurtle:master Sep 30, 2018
@bittner bittner deleted the feature/add-pypi-account-password branch September 30, 2018 16:01
@bittner
Copy link
Member Author

bittner commented Sep 30, 2018

Please wait, until the build is green. Even for seemingly "stupid" changes. 👍

@cool-RR
Copy link
Member

cool-RR commented Sep 30, 2018 via email

@bittner
Copy link
Member Author

bittner commented Sep 30, 2018

tell GitHub "merge this when it's green"?

AFAIK, this is only supported by GitLab CI.

@bittner
Copy link
Member Author

bittner commented Sep 30, 2018

You could now try to put a version tag on a commit (preferably on master) and push it to your repo. Then we would see what happens.

@bittner
Copy link
Member Author

bittner commented Sep 30, 2018

Seems to fail for an authentication error. 😕

Was it the PyPI password of your coolRR user that you encrypted?

@bittner
Copy link
Member Author

bittner commented Sep 30, 2018

... and the publish step was executed even w/o pushing a Git tag? It shouldn't! -- Ah, Travis, you're a pain.

@cool-RR
Copy link
Member

cool-RR commented Sep 30, 2018

Was it the PyPI password of your coolRR user that you encrypted?

Yep.

@cool-RR
Copy link
Member

cool-RR commented Sep 30, 2018

I got a bit lost in the conversation here, if you need me to do something let me know.

@bittner
Copy link
Member Author

bittner commented Oct 1, 2018

Could you try to push a tag to this repository now? e.g.

$ git checkout master
$ git tag 0.3.0
$ git push --tags

I want to see the automatic publishing on PyPI working. Cross your fingers! 🤞

@cool-RR
Copy link
Member

cool-RR commented Oct 2, 2018 via email

@bittner
Copy link
Member Author

bittner commented Oct 2, 2018

Travis says, you didn't do this from the master branch:

Skipping a deployment with the pypi provider because this branch is not permitted: 0.3.0

Can you try again? Ideally, delete the tag you added beforehand.

@cool-RR
Copy link
Member

cool-RR commented Oct 2, 2018 via email

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.

2 participants