Skip to content

ci: run security audit for merge queue - #3245

Merged
CTTY merged 3 commits into
mainfrom
kevinjqliu-require-merge-queue-checks
Sep 17, 2026
Merged

CTTY merged 3 commits into
mainfrom
kevinjqliu-require-merge-queue-checks

Conversation

@kevinjqliu

@kevinjqliu kevinjqliu commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Add security audit as blocker for merge queue.
#3228 was able to be merged even though the "security audit" task failed.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Copilot AI lite review requested due to automatic review settings September 17, 2026 19:09
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@kevinjqliu kevinjqliu changed the title ci: make required merge queue checks reliable ci: run security audit for merge queue Sep 17, 2026

This comment was marked as outdated.

@kevinjqliu
kevinjqliu requested review from CTTY and blackmwk September 17, 2026 19:12
@kevinjqliu

kevinjqliu commented Sep 17, 2026 •

Copy link
Copy Markdown
Contributor Author

needs to rebase #3244

@CTTY CTTY left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@CTTY
CTTY added this pull request to the merge queue Sep 17, 2026
Merged via the queue into main with commit 492234e Sep 17, 2026
24 checks passed
@CTTY
CTTY deleted the kevinjqliu-require-merge-queue-checks branch September 17, 2026 23:39
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.

3 participants