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

github-actions/release: Consolidate #2185

Merged
merged 6 commits into from
Nov 6, 2021

Conversation

jvesely
Copy link
Collaborator

@jvesely jvesely commented Nov 6, 2021

Consolidate release scripts to a single workflow on tag.
Add deployment environments for publishing docs, test-pypi, gh release, and pypi.
Adapt to github-script@v15 changes.
Use TWINE_REPOSITORY_URL env var instead of TWINE_REPOSITORY to avoid needing a config file.
Fix typos and clarify comments.

The environments are created with no restrictions if they don't exist.
Split upload to github release and PyPI.
Add write permission to jobs creating release and uploading artifacts.
Clarify comments and fix typos

Signed-off-by: Jan Vesely <[email protected]>
…OSITORY

The latter needs configuration file to read the url from.

Signed-off-by: Jan Vesely <[email protected]>
github-script@v5 made the change and the release scripts were not
tested.

Signed-off-by: Jan Vesely <[email protected]>
Move all github release handling into one job,
guarded by gh-release deployment

Signed-off-by: Jan Vesely <[email protected]>
@jvesely jvesely added the CI Continuous Integration label Nov 6, 2021
@github-actions
Copy link

github-actions bot commented Nov 6, 2021

This PR causes the following changes to the html docs (ubuntu-latest-3.7-x64):

No differences!

...

See CI logs for the full diff.

@jvesely jvesely merged commit ef014f5 into PrincetonUniversity:devel Nov 6, 2021
@jvesely jvesely deleted the gh-actions branch November 6, 2021 22:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant