Skip to content

Conversation

@lgritz
Copy link
Collaborator

@lgritz lgritz commented Jul 1, 2025

The upload_pypi step only happened for tags, not for other events like PRs. But it would TRY (and fail) if you pushed a v3.0.* tag to your own fork. This patch makes it not even attempt it, except when the tag is pushed to the official repo.

The upload_pypi step only happened for tags, not for other events
like PRs. But it would TRY (and fail) if you pushed a v3.0.* tag to
your own fork. This patch makes it not een attempt it, except when
the tag is pushed to the offical repo.

Signed-off-by: Larry Gritz <[email protected]>
@lgritz lgritz requested a review from zachlewis July 1, 2025 17:16
Copy link
Collaborator

@zachlewis zachlewis left a comment

Choose a reason for hiding this comment

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

Nice 🤘

@lgritz lgritz merged commit cbbb98c into AcademySoftwareFoundation:main Jul 3, 2025
70 checks passed
@lgritz lgritz deleted the lg-wheel branch July 4, 2025 22:10
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jul 5, 2025
…SoftwareFoundation#4820)

The upload_pypi step only happened for tags, not for other events like
PRs. But it would TRY (and fail) if you pushed a v3.0.* tag to your own
fork. This patch makes it not even attempt it, except when the tag is
pushed to the official repo.

Signed-off-by: Larry Gritz <[email protected]>
@lgritz lgritz added the build / testing / port / CI Affecting the build system, tests, platform support, porting, or continuous integration. label Jul 13, 2025
zachlewis pushed a commit to zachlewis/OpenImageIO that referenced this pull request Aug 1, 2025
…SoftwareFoundation#4820)

The upload_pypi step only happened for tags, not for other events like
PRs. But it would TRY (and fail) if you pushed a v3.0.* tag to your own
fork. This patch makes it not even attempt it, except when the tag is
pushed to the official repo.

Signed-off-by: Larry Gritz <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build / testing / port / CI Affecting the build system, tests, platform support, porting, or continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants