Skip to content

Set up CodeQL scanning#29

Merged
jbeckwith-oai merged 1 commit into
mainfrom
codex/setup-codeql
Jul 14, 2026
Merged

Set up CodeQL scanning#29
jbeckwith-oai merged 1 commit into
mainfrom
codex/setup-codeql

Conversation

@jbeckwith-oai

Copy link
Copy Markdown
Contributor

Summary

  • add CodeQL analysis for Go and GitHub Actions workflows
  • run scans on pushes and pull requests targeting main, without a scheduled cron run
  • deny token permissions by default and grant only contents: read and security-events: write to the analysis job
  • pin third-party actions to full commit SHAs

Why

This adds automated code scanning so security issues can be detected when changes are pushed or proposed for main.

Validation

  • go run github.com/rhysd/actionlint/cmd/actionlint@v1.7.7 .github/workflows/codeql.yml
  • YAML parsing
  • git diff --cached --check

@github-advanced-security

Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

@jbeckwith-oai
jbeckwith-oai marked this pull request as ready for review July 14, 2026 19:46
@jbeckwith-oai
jbeckwith-oai requested a review from a team as a code owner July 14, 2026 19:46

@HAYDEN-OAI HAYDEN-OAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

looks good to me. the permissions are appropriately scoped, the action refs are pinned, and both new codeql matrix jobs completed successfully. nice addition!

@jbeckwith-oai
jbeckwith-oai merged commit f982a03 into main Jul 14, 2026
9 checks passed
@jbeckwith-oai
jbeckwith-oai deleted the codex/setup-codeql branch July 14, 2026 20:13
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