Skip to content

Conversation

@MaxDac
Copy link
Contributor

@MaxDac MaxDac commented Dec 3, 2025

CodeQL GitHub action checks are failing consistently in the Omex OSS repo because CodeQL identified two different configurations for the same repository, one detailed in the GH action yaml file, the other in the repository Advanced security itself.

The CodeQL check started failing with this commit.

For more information, check this link: Code scanning tool status · microsoft/Omex.

This PR proposes removing the CodeQL check contained in the yaml file instead of removing the default CodeQL configuration from the Omex OSS repo settings, which can be done without PR.

@MaxDac MaxDac requested a review from a team as a code owner December 3, 2025 15:13
Copilot AI review requested due to automatic review settings December 3, 2025 15:13
@github-actions
Copy link

github-actions bot commented Dec 3, 2025

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code -
Test Code -
Subtotal -
Ignored Code -
Total -

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@github-actions github-actions bot changed the title Removed possibly redundant CodeQL check XS✔ ◾ Removed possibly redundant CodeQL check Dec 3, 2025
Copy link
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 removes the GitHub Actions CodeQL workflow file to resolve a conflict where two different CodeQL configurations existed for the repository (one in the workflow file and one in the repository's Advanced Security settings). The removal allows the repository-level CodeQL configuration to take precedence and eliminates the failing checks.

Key Changes:

  • Complete removal of .github/workflows/codeql-analysis.yml workflow file

@MaxDac MaxDac enabled auto-merge (squash) December 3, 2025 22:19
@MaxDac MaxDac disabled auto-merge December 3, 2025 22:20
@MaxDac MaxDac merged commit 7c05f25 into main Dec 9, 2025
13 checks passed
@MaxDac MaxDac deleted the maxdac/fix/fix-codeql-checks-duplication branch December 9, 2025 16:49
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