Skip to content

Fix CodeQL configuration warning by adding Rust support#99

Merged
jwilger merged 2 commits intomainfrom
fix-codeql-rust-support
Jul 16, 2025
Merged

Fix CodeQL configuration warning by adding Rust support#99
jwilger merged 2 commits intomainfrom
fix-codeql-rust-support

Conversation

@jwilger
Copy link
Copy Markdown
Owner

@jwilger jwilger commented Jul 16, 2025

Summary

  • Added Rust language to CodeQL analysis matrix
  • Updated configuration comments to reflect experimental Rust support
  • UPDATED: Removed custom CodeQL workflow files entirely
  • Fixes GitHub Advanced Security warning about missing Rust configuration

Context

GitHub Advanced Security was showing a warning that CodeQL couldn't determine alerts introduced by PRs because the Rust language configuration was missing.

UPDATE: The build failure revealed that the repository has GitHub's default CodeQL setup enabled, which conflicts with custom workflows. The default setup already provides automatic language detection and support, including Rust.

Solution

Removed the custom CodeQL workflow and configuration files to let the default setup handle code scanning automatically.

Test plan

  • Verify no more CodeQL workflow errors
  • Confirm the warning is resolved in GitHub Advanced Security
  • Default CodeQL setup continues to work

🤖 Generated with Claude Code

- Add 'rust' to the language matrix in CodeQL workflow
- Update comments to reflect that CodeQL now has experimental Rust support as of 2025
- This fixes the GitHub Advanced Security warning about missing configuration

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
Copilot AI review requested due to automatic review settings July 16, 2025 13:40
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

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 adds experimental Rust language support to the CodeQL analysis workflow to resolve a GitHub Advanced Security warning about missing Rust configuration.

  • Added Rust to the CodeQL analysis language matrix alongside JavaScript
  • Created a CodeQL configuration file with appropriate ignore paths and documentation
  • Updated comments to reflect experimental Rust support availability in 2025

Reviewed Changes

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

File Description
.github/workflows/codeql-analysis.yml New CodeQL workflow file with JavaScript and Rust language support
.github/codeql/codeql-config.yml New CodeQL configuration file with ignore paths and Rust support documentation

GitHub's default CodeQL setup is enabled for this repository, which
conflicts with custom workflows. The default setup already provides
Rust support along with other languages automatically.

Removing:
- .github/workflows/codeql-analysis.yml
- .github/codeql/codeql-config.yml

This resolves the build failure and simplifies the security scanning
configuration.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@jwilger jwilger merged commit 6fcac4a into main Jul 16, 2025
10 checks passed
@jwilger jwilger deleted the fix-codeql-rust-support branch July 16, 2025 13:55
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.

2 participants