-
Notifications
You must be signed in to change notification settings - Fork 88
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
Add and document release automation via github actions #155
Conversation
This is likely not working any more, as indicated by for exmaple specifying python 3.5 to be used.
permissions: | ||
# id-token=write is required for pypa/gh-action-pypi-publish, and the PyPI | ||
# project needs to be configured to trust this workflow. | ||
# | ||
# ref: https://github.com/jupyterhub/team-compass/issues/648 | ||
# | ||
id-token: write |
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.
@ryanlovett @yuvipanda @minrk could one of you provide this github workflow with the permissions to publish to PyPI?
You'd visit https://pypi.org/manage/project/jupyter-rsession-proxy/settings/publishing/ and under "Add a new publisher" populate 3/4 fields like:
- owner: jupyterhub
- repo: jupyter-rsession-proxy
- workflow name: release.yaml
- environment: leave unspecified
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.
@consideRatio Done!
7d88195
to
9fbe2fd
Compare
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.
Thanks for adding this automation!
permissions: | ||
# id-token=write is required for pypa/gh-action-pypi-publish, and the PyPI | ||
# project needs to be configured to trust this workflow. | ||
# | ||
# ref: https://github.com/jupyterhub/team-compass/issues/648 | ||
# | ||
id-token: write |
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.
@consideRatio Done!
Thank you @ryanlovett!! |
We had release automation via travis, but it was probably unused as it specified python 3.5 etc. This brings the project up to date with what we find in https://github.com/jupyterhub/jupyterhub-python-repo-template.