Refactor conditionals in action workflows to remove unnecessary syntax#31
Merged
javidahmed64592 merged 1 commit intomainfrom Mar 6, 2026
Merged
Refactor conditionals in action workflows to remove unnecessary syntax#31javidahmed64592 merged 1 commit intomainfrom
javidahmed64592 merged 1 commit intomainfrom
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This pull request updates the conditional syntax used in GitHub Actions workflow and composite action files. The main improvement is switching from the older
${{ ... }}expression syntax to the newer, simplerif: github.event.repository.name == 'template-python'style. This change makes the workflow definitions cleaner and more consistent.Updates to conditional syntax in workflow files:
.github/workflows/build.yml: Changed allifconditions to use the new syntax for bothbuild-wheelandverify-structurejobs..github/workflows/ci.yml: Updatedifconditions for all CI jobs (validate-pyproject, ruff, mypy, pytest, bandit, pip-audit, version-check) to use the new syntax.Updates to conditional syntax in composite actions:
.github/actions/build/build-wheel/action.yml: Updatedifconditions for installing Python core dependencies..github/actions/build/verify-structure/action.yml: Updatedifconditions for installing Python core dependencies..github/actions/ci/bandit/action.yml,.github/actions/ci/mypy/action.yml,.github/actions/ci/pip-audit/action.yml,.github/actions/ci/pytest/action.yml,.github/actions/ci/ruff/action.yml,.github/actions/ci/validate-pyproject/action.yml,.github/actions/ci/version-check/action.yml: Updatedifconditions for installing Python dev dependencies. [1] [2] [3] [4] [5] [6] [7]Updates to setup actions:
.github/actions/setup/install-python-core/action.yml,.github/actions/setup/install-python-dev/action.yml: Updatedifconditions for installing UV Python dependencies. [1] [2]