Skip to content

Add manual trigger to deploy to pypi#2192

Merged
abidlabs merged 10 commits into
gradio-app:mainfrom
abidlabs:main
Sep 6, 2022
Merged

Add manual trigger to deploy to pypi#2192
abidlabs merged 10 commits into
gradio-app:mainfrom
abidlabs:main

Conversation

@abidlabs
Copy link
Copy Markdown
Member

@abidlabs abidlabs commented Sep 6, 2022

This creates a "run workflow" button for the "deploy to pypi" action, similar to this:

image

I haven't been able to fully test the deployment on my fork because secrets, etc. don't transfer over to the fork, but I can confirm that the button appears. I'll do a complete test once this is merged in.

Also removes the whole "merge to a branch starting with 'release'" to deploy a test branch as it's quite cumbersome and we won't need it if this works

Fixes: #2190

Comment thread .github/workflows/deploy-pypi.yml
Copy link
Copy Markdown
Collaborator

@freddyaboulton freddyaboulton left a comment

Choose a reason for hiding this comment

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

Thanks @abidlabs ! Only pending change is checking whether or not the version exists via pypi in the github action.

@abidlabs
Copy link
Copy Markdown
Member Author

abidlabs commented Sep 6, 2022

Added the pypi check near the beginning of the GitHub action. Thanks for the suggestion @freddyaboulton, and this should be good to go now.

@abidlabs abidlabs merged commit 3464673 into gradio-app:main Sep 6, 2022
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.

Add manual trigger for releases to pypi

2 participants