-
Notifications
You must be signed in to change notification settings - Fork 23
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
Enable PyPI Publishing with CircleCI #49
Conversation
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.
Happy to re-review if you want to address the nits
command: | | ||
python -m venv venv || virtualenv venv | ||
. venv/bin/activate | ||
python setup.py verify |
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.
nice
setup.py
Outdated
description = 'verify that the git tag matches our version' | ||
|
||
def run(self): | ||
tag = os.getenv('CIRCLE_TAG') |
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.
nit: should we bail out with a different message if tag is None
?
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.
Aye. This depends on the script being run on circle
on a tagged branch. I can add some sanity.
@borgstrom - nits addressed and testing completed. Test PyPi is a bit slow to remove versions/projects that I want to re-use, but the api calls are happening. I'd expect us to have no issues publishing |
#48
We would like to make
terraformpy
available publicly on PyPi. In order to do this, I've updated our CircleCI workflow to include apublish
job.This job will be triggered when a tag like
v1.1.1
is created, verifying that ourVERSION
file matches the tagged version and publishing to: https://pypi.org/project/terraformpy/#description