Skip to content

Fix accessing repo id in the release workflow #2210

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

Merged
merged 3 commits into from
Jul 30, 2025

Conversation

webknjaz
Copy link
Member

This patch proxies the ${{ env }} context access in the conditional
clause for the publish-pypijob through thebuildjoboutputs`.

This is necessary since said context is not available in some parts of
the GitHub Actions CI/CD workflow defintions.

The change set also includes running the build job in PRs.

Contributor checklist
  • Included tests for the changes.
  • A change note is created in changelog.d/ (see changelog.d/README.md for instructions) or the PR text says "no changelog needed".
Maintainer checklist
  • If no changelog is needed, apply the skip-changelog label.
  • Assign the PR to an existing or new milestone for the target version (following Semantic Versioning).

webknjaz added 2 commits July 30, 2025 15:23
This patch proxies the `${{ env }}` context access in the conditional
`clause for the `publish-pypi` job through the `build` job `outputs`.

This is necessary since said context is not available in some parts of
the GitHub Actions CI/CD workflow defintions.
@webknjaz webknjaz requested a review from sirosen July 30, 2025 13:27
@webknjaz webknjaz added this pull request to the merge queue Jul 30, 2025
Merged via the queue into jazzband:main with commit 1c7d9fb Jul 30, 2025
43 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants