Skip to content

Group CodeQL action updates in one Dependabot PR - #316

Merged
arifBurakDemiray merged 1 commit into
masterfrom
dependabot-group-codeql
Oct 22, 2026
Merged

arifBurakDemiray merged 1 commit into
masterfrom
dependabot-group-codeql

Conversation

@arifBurakDemiray

@arifBurakDemiray arifBurakDemiray commented Oct 22, 2026 •

Copy link
Copy Markdown
Member

What

Adds a codeql group (pattern github/codeql-action*) to the GitHub Actions entry in .github/dependabot.yml.

Why

Dependabot opened separate PRs for github/codeql-action/init (#313) and github/codeql-action/analyze (#314). CodeQL fails when the two steps run different versions ("Loaded a configuration file for version '4.39.0', but running version '4.38.2'. All steps in a workflow that use github/codeql-action must use the same version"), so each PR failed on its own. With the group, both arrive in one PR. #313 now carries both bumps for this round.

Dependabot reads its configuration from the default branch, so #316 (master) makes it take effect, and #317 (staging) keeps the file identical so the next release merge does not conflict.

Verification

  • The file parses, and the GitHub Actions entry has groups: {codeql: {patterns: [github/codeql-action*]}} with target-branch: staging unchanged.
  • Identical on both branches.

🤖 Generated with Claude Code

Dependabot opened separate PRs for github/codeql-action/init and
github/codeql-action/analyze (#313, #314). The analyze step fails when the
two run different versions, so each PR failed CodeQL on its own. A group
makes them one PR.

Dependabot reads this file from master; staging carries the same file so
the next release merge does not conflict.
@github-actions

Copy link
Copy Markdown

Dependency security scan

Scanned 24 resolved dependencies, failing on HIGH and above.

No known vulnerabilities in 24 resolved dependencies.

Result: passed

Full run log

@github-actions

Copy link
Copy Markdown

Unit Test Results 🚀

611 tests  ±0   611 ✅ ±0   2m 5s ⏱️ -1s
 54 suites ±0     0 💤 ±0 
 54 files   ±0     0 ❌ ±0 

Results for commit b31ec35. ± Comparison against base commit be77c13.

@arifBurakDemiray
arifBurakDemiray marked this pull request as ready for review October 22, 2026 06:15
@arifBurakDemiray
arifBurakDemiray merged commit 3d5beb9 into master Oct 22, 2026
10 checks passed
@github-actions

Copy link
Copy Markdown

Code coverage

module lines branches
sdk-java 90.4% (5092/5634) 81.1% (2122/2617)
sdk-java-ui 70.5% (968/1374) 63.2% (499/790)
Least covered classes (10 shown)
class module lines covered uncovered
ly.count.sdk.java.ui.JavaFxContentDisplay sdk-java-ui 10.6% (23/216) 193
ly.count.sdk.java.ui.FxSurfaces sdk-java-ui 74.8% (172/230) 58
ly.count.sdk.java.ui.ExternalBrowser sdk-java-ui 55.0% (66/120) 54
ly.count.sdk.java.ui.JavaFxWidgetHost sdk-java-ui 81.2% (190/234) 44
ly.count.sdk.java.internal.SDKCore sdk-java 87.9% (304/346) 42
ly.count.sdk.java.internal.ModuleContent sdk-java 86.8% (210/242) 32
ly.count.sdk.java.internal.TimedEvents sdk-java 60.8% (48/79) 31
ly.count.sdk.java.internal.SessionImpl sdk-java 89.4% (262/293) 31
ly.count.sdk.java.internal.SDKStorage sdk-java 84.3% (150/178) 28
ly.count.sdk.java.internal.ModuleFeedback sdk-java 86.7% (176/203) 27

Full run log

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.

1 participant