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

Create python-publish.yml #391

Merged
merged 1 commit into from
May 17, 2023
Merged

Create python-publish.yml #391

merged 1 commit into from
May 17, 2023

Conversation

ericspod
Copy link
Member

This introduces the script to publish a PyPI package whenever a new release is made on the Github repo. When a new release is made (ie. not when a commit a tagged) the PyPI key is used by the action to push the new package, pre-releases should work as well.

This introduces the script to publish a PyPI package whenever a new release is made on the Github repo. When a new release is made (ie. not when a commit a tagged) the PyPI key is used by the action to push the new package, pre-releases should work as well. 

Signed-off-by: Eric Kerfoot <17726042+ericspod@users.noreply.github.com>
@ericspod ericspod requested review from Warvito and Nic-Ma May 16, 2023 12:57
@ericspod
Copy link
Member Author

We could modify the script to include verification such as what I have here.

@ericspod
Copy link
Member Author

The PYPI_API_TOKEN secret value needs to be added to the repo, @Warvito would you have that?

@Nic-Ma Nic-Ma requested a review from mingxin-zheng May 17, 2023 08:25
@ericspod ericspod merged commit 01a4654 into main May 17, 2023
@ericspod ericspod deleted the ericspod-pypi branch May 17, 2023 13:26
@Nic-Ma
Copy link
Contributor

Nic-Ma commented May 17, 2023

Hi @ericspod ,

Could you please help also update the installation doc for Pypi?
https://github.com/Project-MONAI/GenerativeModels#installation

Thanks in advance.

@ericspod
Copy link
Member Author

Hi @Nic-Ma, I will and the docs in the tutorials repo, however we are working on getting the key first and making sure the PyPI package is correct.

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.

None yet

3 participants