Skip to content

Add mu_crypto_release to CodeQL workflow sync#567

Merged
Flickdm merged 1 commit intomicrosoft:mainfrom
Flickdm:feat/codeql-mu-crypto-release
May 1, 2026
Merged

Add mu_crypto_release to CodeQL workflow sync#567
Flickdm merged 1 commit intomicrosoft:mainfrom
Flickdm:feat/codeql-mu-crypto-release

Conversation

@Flickdm
Copy link
Copy Markdown
Member

@Flickdm Flickdm commented May 1, 2026

Add microsoft/mu_crypto_release to the leaf CodeQL CI workflow sync list so the repo receives the codeql.yml workflow and gets CodeQL security scanning on push and PR.

Add microsoft/mu_crypto_release to the leaf CodeQL CI workflow
sync list so the repo receives the codeql.yml workflow and gets
CodeQL security scanning on push and PR.

Signed-off-by: Doug Flick <dougflick@microsoft.com>
@Flickdm Flickdm force-pushed the feat/codeql-mu-crypto-release branch from 9cc2bad to 9bec65e Compare May 1, 2026 14:52
@Flickdm Flickdm merged commit 2ffad0f into microsoft:main May 1, 2026
2 checks passed
@Flickdm Flickdm deleted the feat/codeql-mu-crypto-release branch May 1, 2026 17:07
Comment thread .sync/Files.yml
repos: |
microsoft/mu_basecore
microsoft/mu_common_intel_min_platform
microsoft/mu_crypto_release
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Have you tested this in mu_crypto_release already?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

No should I beforehand? Are you concerned about churn?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, we check that everything works before sync. Otherwise, the process would be more chaotic than it typically is as small adjustments are often needed.

Could you try the workflow on your fork real quick before the file sync happens?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@Flickdm, the CodeQL workflow does fail in mu_crypto_release (OneCryptoPkg example) - microsoft/mu_crypto_release#245:

image

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Limited to X64 for mu_crypto_release in #571.

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.

4 participants