Skip to content
This repository was archived by the owner on Aug 15, 2025. It is now read-only.

Conversation

seemethere
Copy link
Member

Signed-off-by: Eli Uriegas eliuriegas@fb.com

Signed-off-by: Eli Uriegas <eliuriegas@fb.com>
Copy link
Contributor

@walterddr walterddr left a comment

Choose a reason for hiding this comment

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

left one question, but overall lgtm

promote_conda torchcsprng conda "${TORCHCSPRNG_VERSION}"

# Uncomment these to promote to pypi
# promote_pypi torch "${PYTORCH_VERSION}"
Copy link
Contributor

Choose a reason for hiding this comment

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

would this script be used inside pytorch/pytorch/.circleci? I was wondering if we can just use env var to control this?

Copy link
Contributor

Choose a reason for hiding this comment

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

Or maybe a command line argument

Copy link
Member Author

Choose a reason for hiding this comment

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

No we typically promotion is the last manual step in the release process.

As part of this, we make it purposefully difficult to promote to PyPI since it cannot be undone.

@seemethere seemethere merged commit f964d9d into pytorch:master Sep 23, 2020
@seemethere seemethere deleted the add_promo_script branch September 23, 2020 18:14
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants