Skip to content

disable codeql#1851

Merged
breedx-splk merged 2 commits into
open-telemetry:mainfrom
breedx-splk:disable_codeql_job
Jul 2, 2026
Merged

disable codeql#1851
breedx-splk merged 2 commits into
open-telemetry:mainfrom
breedx-splk:disable_codeql_job

Conversation

@breedx-splk

Copy link
Copy Markdown
Contributor

Every PR build now shows an ❌ in the favicon and in the status checks, because we broke CodeQL when we merged #1786. CodeQL does not yet support kotlin 2.4.0, so until it does, all of our builds will be "broken". CodeQL isn't in the critical path, but it's obnoxious to have all these false failures everywhere.

So let's disable it for now by renaming the workflow, so that we can easily rename it back.

Upstream tracking issue is here: github/codeql#21938

@breedx-splk breedx-splk requested a review from a team as a code owner July 2, 2026 21:07
Copilot AI review requested due to automatic review settings July 2, 2026 21:07

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.

Copilot wasn't able to review any files in this pull request.

@codecov

codecov Bot commented Jul 2, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.50%. Comparing base (4da029e) to head (1297987).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1851   +/-   ##
=======================================
  Coverage   64.50%   64.50%           
=======================================
  Files         166      166           
  Lines        3648     3648           
  Branches      386      386           
=======================================
  Hits         2353     2353           
  Misses       1181     1181           
  Partials      114      114           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bidetofevil bidetofevil 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.

LGTM

@breedx-splk breedx-splk enabled auto-merge (squash) July 2, 2026 22:32
@breedx-splk breedx-splk merged commit a665824 into open-telemetry:main Jul 2, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants