Skip to content
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

.sync/workflows/codeql: Use Version.njk cargo-make version #323

Merged

Conversation

makubacki
Copy link
Member

Most of the workflow can stay intact with GitHub caching working
as-is but the cargo-make version now needs to match the version that
is being synced to repos (i.e. matches rust-toolchain.toml).


Note: We can consider using cargo binstall in the future but right now
only cargo-make is needed, and it is simple to directly cache that binary
as-is with existing code.

@makubacki makubacki added the semver:patch Pull requests that should increment the release patch version label Feb 23, 2024
@makubacki makubacki self-assigned this Feb 23, 2024
Most of the workflow can stay intact with GitHub caching working
as-is but the cargo-make version now needs to match the version that
is being synced to repos (i.e. matches rust-toolchain.toml).

Signed-off-by: Michael Kubacki <[email protected]>
@makubacki makubacki force-pushed the use_sync_cargo_make_codeql_workflows branch from a51a263 to 9fe0213 Compare February 23, 2024 22:38
@makubacki makubacki merged commit d4b21fc into microsoft:main Feb 23, 2024
3 checks passed
makubacki added a commit to makubacki/mu_devops that referenced this pull request Feb 26, 2024
Changes since last release:
microsoft/mu_devops@v9.1.5...v9.1.6

General release info: https://github.com/microsoft/mu_devops/releases

---

Includes:

- Sync RustSetupSteps.yml to use cargo-binstall to install cargo binary tools (microsoft#320)
- Update RustSetupSteps.yml (microsoft#321)
- Repo File Sync: synced file(s) with microsoft/mu_devops (microsoft#322)
- .sync/workflows/codeql: Use Version.njk cargo-make version (microsoft#323)

In particular, including microsoft#320 in this tag allows downstream repos to use the changes.

Signed-off-by: Michael Kubacki <[email protected]>
makubacki added a commit that referenced this pull request Feb 26, 2024
Changes since last release:
v9.1.5...v9.1.6

General release info: https://github.com/microsoft/mu_devops/releases

---

Includes:

- Sync RustSetupSteps.yml to use cargo-binstall to install cargo binary
tools (#320)
- Update RustSetupSteps.yml (#321)
- Repo File Sync: synced file(s) with microsoft/mu_devops (#322)
- .sync/workflows/codeql: Use Version.njk cargo-make version (#323)

In particular, including #320 in this tag allows downstream repos to use
the changes.

Signed-off-by: Michael Kubacki <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch Pull requests that should increment the release patch version
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants