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

Update workflow permissions #2718

Merged
merged 1 commit into from
Jan 24, 2025
Merged

Update workflow permissions #2718

merged 1 commit into from
Jan 24, 2025

Conversation

kaeluka
Copy link
Contributor

@kaeluka kaeluka commented Jan 24, 2025

Related issue: https://github.com/github/codeql-core/issues/4779

This adds a minimal permissions block to query-filters.yml. The CI check seems to still pass.

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

@Copilot Copilot bot review requested due to automatic review settings January 24, 2025 10:54
@kaeluka kaeluka requested a review from a team as a code owner January 24, 2025 10:54

Choose a reason for hiding this comment

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

Copilot reviewed 1 out of 1 changed files in this pull request and generated no comments.

Tip: Copilot code review supports C#, Go, Java, JavaScript, Markdown, Python, Ruby and TypeScript, with more languages coming soon. Learn more

@kaeluka kaeluka marked this pull request as draft January 24, 2025 10:54
@kaeluka kaeluka removed the request for review from a team January 24, 2025 10:55
@kaeluka kaeluka marked this pull request as ready for review January 24, 2025 11:31
@kaeluka kaeluka merged commit 573ad88 into main Jan 24, 2025
505 checks passed
@kaeluka kaeluka deleted the kaeluka/4779-1 branch January 24, 2025 13:16
@github-actions github-actions bot mentioned this pull request Jan 24, 2025
8 tasks
# 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