PR checks: small fixups to `prepare-action` by angelapwen · Pull Request #2548 · github/codeql-action · GitHub

While working on a version of this for testing on runner images, I noticed some improvements to be made:

the action now expects stable CLI version inputs as stable-vX.Y.Z rather than with dates. Our current PR checks already use this format but the documentation for the input needed to be updated.
the nightly version format also should expect nightly-YYYYMMDD rather than with dashes
the nightly bundle URL formatting has been corrected to remove the manual suffix — eg. the current nightly URLs look like: https://github.com/dsp-testing/codeql-cli-nightlies/releases/download/codeql-bundle-20240823/codeql-bundle-linux64.tar.gz. We haven't caught this because none of our PR checks use the dated nightly bundles.
when the version is linked or default we return the appropriate tools input early.

…on input

angelapwen added 2 commits October 15, 2024 15:53

PR checks: fix formatting for nightly bundle URL and stable CLI versi…

ded11c6

…on input

Return early if version is linked or default

2f0f924

angelapwen force-pushed the angelapwen/fix-prepare-test branch from 9092cbf to 2f0f924 Compare October 15, 2024 23:17

angelapwen marked this pull request as ready for review October 15, 2024 23:28

angelapwen requested a review from a team as a code owner October 15, 2024 23:28

henrymercer approved these changes Oct 16, 2024

View reviewed changes

angelapwen merged commit ad94f2f into main Oct 16, 2024

angelapwen deleted the angelapwen/fix-prepare-test branch October 16, 2024 15:27

This was referenced Oct 22, 2024

Merge main into releases/v3 #2561

Merged

Merge releases/v3 into releases/v2 #2563

Merged

PR checks: small fixups to prepare-action - #2548