Skip to content

POLY-115: use relative paths for other workflows in this repo - #182

Merged
tmthrgd-aligent merged 3 commits into
mainfrom
fix/POLY-115_use_relative_paths_for_uses
Sep 16, 2026
Merged

tmthrgd-aligent merged 3 commits into
mainfrom
fix/POLY-115_use_relative_paths_for_uses

Conversation

@tmthrgd-aligent

@tmthrgd-aligent tmthrgd-aligent commented Sep 15, 2026 •

Copy link
Copy Markdown
Contributor

One of the consumers of our workflows have turned on the setting that requires all workflows to be pinned, this extends into the workflows in this repo. According to the GitHub Docs, $/.github/workflows/{filename} is the syntax to use to reference workflows in the same repo. Hopefully this works with the .github/actions directory the same way.

This is better than what we had regardless of client need as it ensures our workflows don't mismatch the version of the workflows or actions they depend on.

One of the consumers of our workflows have turned on the setting that
requires all workflows to be pinned, this extends into the workflows in
this repo. According to the GitHub Docs, $/.github/workflows/{filename}
is the syntax to use to reference workflows in the same repo. Hopefully
this works with the .github/actions directory the same way.

This is better than what we had regardless of client need as it ensures
our workflows don't mismatch the version of the workflows or actions
they depend on.
This syntax was added by GitHub on July 30th and actionlint has not been
updated to support it. This is a valid and working syntax so ignore the
errors.

See rhysd/actionlint#711.
@tmthrgd-aligent
tmthrgd-aligent force-pushed the fix/POLY-115_use_relative_paths_for_uses branch from ca457ba to 889c8e6 Compare September 15, 2026 07:11
We need to update the version of zizmorcore/zizmor-action to support the
new $/ actions repo-reference syntax. While we are here, update all the
actions to their latest versions.

@aaronmedina-dev aaronmedina-dev left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@tmthrgd-aligent

Copy link
Copy Markdown
Contributor Author

@tmthrgd-aligent
tmthrgd-aligent merged commit eec4107 into main Sep 16, 2026
3 checks passed
@tmthrgd-aligent
tmthrgd-aligent deleted the fix/POLY-115_use_relative_paths_for_uses branch September 16, 2026 02:07
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.

2 participants