Skip to content

Replace autobuild with explicit Maven build in CodeQL workflow - #557

Merged
thomasturrell merged 4 commits into
mainfrom
copilot/remove-autobuild-step
Aug 14, 2026
Merged

Replace autobuild with explicit Maven build in CodeQL workflow#557
thomasturrell merged 4 commits into
mainfrom
copilot/remove-autobuild-step

Conversation

Copilot AI commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

github/codeql-action/autobuild doesn't reliably detect Maven projects. Replacing it with an explicit ./mvnw clean verify -DskipTests ensures CodeQL always analyses a properly built artifact.

Changes

  • .github/workflows/codeql.yml
    • Remove autobuild step
    • Add explicit Maven build step: ./mvnw clean verify -DskipTests
    • Remove stale commented-out manual build instructions

Checklist:

  • Public methods are documented
  • Public methods are tested
  • New and existing tests pass when run locally
  • There are no new warnings or errors

Co-authored-by: thomasturrell <1552612+thomasturrell@users.noreply.github.com>
Copilot AI changed the title [WIP] Remove autobuild step from CodeQL workflow Replace autobuild with explicit Maven build in CodeQL workflow Aug 14, 2026
Copilot AI requested a review from thomasturrell August 14, 2026 11:20
@thomasturrell
thomasturrell marked this pull request as ready for review August 14, 2026 11:42
Copilot AI lite review requested due to automatic review settings August 14, 2026 11:42

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

This PR updates the repository’s CodeQL GitHub Actions workflow to avoid github/codeql-action/autobuild, which can be unreliable for Maven multi-module projects, and instead runs an explicit Maven wrapper build so CodeQL analyzes compiled artifacts consistently.

Changes:

  • Removed the github/codeql-action/autobuild step from the CodeQL workflow.
  • Added an explicit Maven build step (./mvnw clean verify -DskipTests) prior to analysis.
  • Removed stale commented-out manual build instructions.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread .github/workflows/codeql.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
…ze steps

Co-authored-by: thomasturrell <1552612+thomasturrell@users.noreply.github.com>
auto-merge was automatically disabled August 14, 2026 12:17

Head branch was pushed to by a user without write access

@thomasturrell
thomasturrell enabled auto-merge (squash) August 14, 2026 12:29
@sonarqubecloud

Copy link
Copy Markdown

@thomasturrell
thomasturrell merged commit e96871d into main Aug 14, 2026
4 checks passed
@thomasturrell
thomasturrell deleted the copilot/remove-autobuild-step branch August 14, 2026 12:30
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