Skip to content

PYTHON-5877 Add codeql composite action - #108

Merged
aclark4life merged 8 commits into
mongodb-labs:mainfrom
aclark4life:PYTHON-5877
Jul 8, 2026
Merged

aclark4life merged 8 commits into
mongodb-labs:mainfrom
aclark4life:PYTHON-5877

Conversation

@aclark4life

@aclark4life aclark4life commented Jun 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Adds a codeql composite action that encapsulates the full CodeQL analysis workflow (checkout, Python setup, init, optional manual build, analyze) for a single language.
  • Centralises the pinned github/codeql-action hash so Dependabot only needs to update one file across all drivers instead of every repo independently.

Test Plan

  • Pre-commit hooks (Validate GitHub Actions) passed on commit.
  • The composite action is consumed by the companion mongo-python-driver PR (PYTHON-5877), which runs the existing CodeQL workflow unchanged end-to-end.

Centralises the CodeQL analysis steps and pinned action version so
Dependabot only needs to update one file for all drivers.
Comment thread README.md Resolved
Comment thread README.md Resolved
@aclark4life
aclark4life marked this pull request as ready for review July 1, 2026 20:08
@aclark4life
aclark4life requested review from blink1073 and Copilot July 1, 2026 20:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds a new codeql composite GitHub Action to standardize CodeQL scanning across driver repos by centralizing the pinned github/codeql-action reference, and documents its usage in the repository README.

Changes:

  • Introduces codeql/action.yml composite action to run CodeQL init/analyze (with optional manual build).
  • Documents the new action and provides workflow examples in README.md.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
README.md Adds documentation and example workflows for consuming the new codeql composite action.
codeql/action.yml Adds a composite action that checks out code, sets up language tooling, initializes CodeQL, optionally builds, and runs analysis.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread README.md Resolved
Comment thread README.md Resolved
@aclark4life
aclark4life requested a review from Copilot July 1, 2026 20:19
Comment thread codeql/action.yml Fixed
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

Comment thread README.md Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Fixed
Comment thread codeql/action.yml Fixed
aclark4life and others added 3 commits July 7, 2026 11:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

Comment thread README.md Resolved
Comment thread README.md Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread codeql/action.yml Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Comment thread codeql/action.yml Resolved
Comment thread codeql/action.yml Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

Comment thread README.md Resolved

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

@aclark4life

Copy link
Copy Markdown
Contributor Author

@blink1073 GTG according to the 🤖 s and I

@blink1073

Copy link
Copy Markdown
Member

FYI The codeql job for pymongo-auth-aws is failing at startup, so I'm going to fix that immediately using this action as part of greenerbuild.

@blink1073 blink1073 left a comment

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.

LGTM

@aclark4life
aclark4life merged commit 95f9ce9 into mongodb-labs:main Jul 8, 2026
6 checks passed
@aclark4life
aclark4life deleted the PYTHON-5877 branch July 8, 2026 17:24
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