Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@vyasr
Copy link
Contributor

@vyasr vyasr commented Sep 25, 2022

Description

This PR adds in Github Actions workflows to deploy packages to TestPyPI on release branches and to PyPI itself whenever a tag is pushed.

Motivation and Context

We need these in place for releases. When switching from CircleCI to Github Actions we chose not to port the deployment step, opting to instead wait until we had an actual release since we knew we would need to work through some issues. This PR handles those changes.

Checklist:

@vyasr vyasr requested a review from lyrivera September 25, 2022 03:48
@codecov
Copy link

codecov bot commented Sep 25, 2022

Codecov Report

Merging #821 (cd2e6d3) into master (85a122e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #821   +/-   ##
=======================================
  Coverage   78.46%   78.46%           
=======================================
  Files          64       64           
  Lines        7230     7230           
  Branches     1465     1465           
=======================================
  Hits         5673     5673           
  Misses       1244     1244           
  Partials      313      313           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@vyasr
Copy link
Contributor Author

vyasr commented Sep 25, 2022

If reviewers are interested you can look into the actions tab to see the last few runs of both the TestPyPI and normal PyPI upload workflows. Up until the last few commits I had it triggering on every push to the branch and disabled the final twine uploads, so you can see the rest of the output there. You can also see that in the two cases where I did enable TestPyPI uploads we did get new packages (dev0 and dev1 releases).

I tested it reasonably thoroughly and feel pretty good with it, but we can of course do some final testing once that is merged by rebasing our 1.8 release branch after this PR is merged and making any final edits in that PR.

@vyasr vyasr mentioned this pull request Sep 25, 2022
@bdice
Copy link
Member

bdice commented Sep 26, 2022

@vyasr Thank you so much for working on this!

@vyasr vyasr requested a review from bdice September 26, 2022 23:55
Copy link
Member

@b-butler b-butler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Besides @bdice's comments the config looks good to me.

Copy link
Contributor

@csadorf csadorf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for taking this on! Two suggestions.

b-butler and others added 3 commits October 4, 2022 10:29
Changes the publish-packages action to use only one `pip install` command.

Co-authored-by: Carl Simon Adorf <[email protected]>
Also separate building and publishing into separate jobs.
@b-butler b-butler requested review from b-butler and csadorf October 4, 2022 14:53
@b-butler
Copy link
Member

b-butler commented Oct 4, 2022

@bdice, @csadorf, @vyasr I went ahead and applied all suggested changes. I do not know how to test this without merging and updating release/1.8.0, but let me know if you have any more changes.

Copy link
Member

@bdice bdice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm going to remove trigger-ability (see comments). I'd like a second approval from another maintainer, then we can merge.

edit: I'm going to merge this now, and then we can test it directly in the release branch.

@bdice bdice enabled auto-merge (squash) October 5, 2022 04:05
@bdice bdice merged commit c9a48fc into master Oct 5, 2022
@bdice bdice deleted the feature/actions_deploy_pypi branch October 5, 2022 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants