Skip to content

chore(ci): expand dependabot coverage and add Java/Android CI#66

Merged
ksroda-sa merged 5 commits into
mainfrom
chore/expand-ci-coverage
May 6, 2026
Merged

chore(ci): expand dependabot coverage and add Java/Android CI#66
ksroda-sa merged 5 commits into
mainfrom
chore/expand-ci-coverage

Conversation

@ksroda-sa
Copy link
Copy Markdown
Collaborator

Summary

  • Close dependabot coverage gaps: add npm entries for vue/* and node/*, maven entries for java/*, nuget entries for dotnet/* — these frameworks were never receiving automated bumps
  • Add test-java.yml so maven bumps have a build+test gate before auto-merge (Temurin JDK 21, mvn -B verify, auto-discovers every pom.xml)
  • Add test-android.yml building RN Android assembleDebug, so RN bumps have a native build gate even though they remain manual-review
  • Exclude /samples/react-native/* from dependabot auto-merge — tsc --noEmit can't catch native autolinking / API-shape breakage

Coverage matrix after this PR

Framework Dep bumps CI gate Auto-merge
react / angular / vue / node npm test-js patch+minor
react-native npm test-js + test-android manual review (intentional)
java maven test-java (new) patch+minor
dotnet nuget test-dotnet patch+minor

🤖 Generated with Claude Code

…workflows

Close gaps where dep bumps were either never proposed or merged without a
CI gate:

- dependabot.yml: add npm entries for vue/* and node/*, maven entries for
  java/*, nuget entries for dotnet/*
- test-java.yml: new workflow auto-discovers pom.xml, runs `mvn -B verify`
  on JDK 21 — gates maven bumps before auto-merge
- test-android.yml: new workflow builds RN Android (assembleDebug) so RN
  bumps have a native build gate even though they're manual-review only
- dependabot-auto-merge.yml: exclude /samples/react-native/* — RN bumps
  can change native autolinking that `tsc --noEmit` doesn't catch

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Previously every workflow triggered on `samples/**`, so a bump in one
framework rebuilt every other framework's matrix. Path filters now match
only the framework each workflow tests:

- test-android: samples/react-native/** + samples/android/**
- test-java: samples/java/**
- test-dotnet: samples/dotnet/**
- test-js: samples/{react,angular,vue,node,react-native}/**

Each workflow also re-triggers when its own file changes.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

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

Expands Dependabot coverage across additional sample frameworks (Vue/Node/Java/.NET) and introduces new CI workflows to gate dependency updates with Java Maven verification and React Native Android builds, while preventing auto-merge for React Native updates.

Changes:

  • Add test-java.yml to discover Maven projects under samples/** and run mvn -B verify on each.
  • Add test-android.yml to discover Android Gradle wrapper projects under samples/** and run ./gradlew assembleDebug (including JS dependency install for RN samples).
  • Update Dependabot + auto-merge rules to cover new ecosystems/directories and exclude React Native directories from patch/minor auto-merge.

Reviewed changes

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

File Description
.github/workflows/test-java.yml New CI workflow to matrix-discover pom.xml projects under samples/** and run Maven verify.
.github/workflows/test-android.yml New CI workflow to matrix-discover Android projects under samples/** and build assembleDebug (RN-aware setup).
.github/workflows/dependabot-auto-merge.yml Excludes Dependabot PRs targeting /samples/react-native/** from patch/minor auto-approval/auto-merge.
.github/dependabot.yml Adds Dependabot entries for Vue/Node/React Native (manual review), Maven (Java), and NuGet (.NET) sample directories.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/test-android.yml Outdated
Comment thread .github/dependabot.yml Outdated
Comment thread .github/workflows/test-java.yml
ksroda-sa and others added 3 commits May 4, 2026 14:29
OpenSAML 4.x (transitive dep of spring-security-saml2-service-provider)
is hosted on build.shibboleth.net, not Maven Central — without this repo
declared, dependency resolution fails on a clean cache (e.g. CI).

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- test-android.yml: install Android SDK 36 / build-tools 36.0.0 to match
  the RN samples' compileSdkVersion = 36 (was 35, would fail clean runner)
- dependabot.yml: clarify that RN auto-merge exclusion is directory-based
  in the auto-merge workflow, not label-based

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@ksroda-sa ksroda-sa marked this pull request as ready for review May 4, 2026 12:44
@ksroda-sa ksroda-sa requested review from a team as code owners May 4, 2026 12:44
@ksroda-sa ksroda-sa merged commit 5f072f7 into main May 6, 2026
26 checks passed
@ksroda-sa ksroda-sa deleted the chore/expand-ci-coverage branch May 6, 2026 07:59
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.

4 participants