Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

limit codeQL push trigger to main branch #1507

Merged
merged 2 commits into from
Feb 13, 2025
Merged

Conversation

spencerschrock
Copy link
Member

I noticed the codeQL action running twice on a PR I updated, once because of the pull_request trigger, and once because of the push trigger. This matches what the starter workflow has:

https://github.com/actions/starter-workflows/blob/a00915e13e33549ccc21799315b83d80afa2757c/code-scanning/codeql.yml#L14-L18

I noticed the codeQL action running twice on a PR I updated, once
because of the pull_request trigger, and once because of the push
trigger.

Signed-off-by: Spencer Schrock <sschrock@google.com>
@spencerschrock spencerschrock requested review from a team, justaugustus and raghavkaul and removed request for a team February 10, 2025 22:21
@justaugustus justaugustus enabled auto-merge (squash) February 13, 2025 01:57
Copy link
Member

@justaugustus justaugustus left a comment

Choose a reason for hiding this comment

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

Good catch, @spencerschrock!

@justaugustus justaugustus merged commit ae7548a into ossf:main Feb 13, 2025
9 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants