Skip to content

feat(gradle-build-pr): add run-tests flag for BOM and test-less projects#10

Merged
TheMeinerLP merged 3 commits into
mainfrom
claude/fix-gradle-test-config-JT1yW
May 13, 2026
Merged

feat(gradle-build-pr): add run-tests flag for BOM and test-less projects#10
TheMeinerLP merged 3 commits into
mainfrom
claude/fix-gradle-test-config-JT1yW

Conversation

@TheMeinerLP
Copy link
Copy Markdown
Contributor

Summary

  • Neuer Boolean-Input run-tests (default true) am gradle-build-pr-Workflow, damit dasselbe reusable Workflow für drei Projekt-Klassen funktioniert: Projekte mit Tests, ohne Tests und reine BOMs (java-platform).
  • Wenn run-tests: false: der JUnit-Upload-Step und der test-report-Aggregations-Job werden geskippt — kein leerer Check, kein „No test results found"-PR-Comment mehr.
  • gradle-task wird zur Laufzeit aus inputs.gradle-task (falls gesetzt), sonst aus run-tests aufgelöst (build test bei true, build bei false). Default-Input ist jetzt leer; explizite Werte überschreiben die Auflösung weiterhin.
  • README erweitert um BOM-Beispiel und Hinweis im „Defaults at a glance"-Block.

Bestehende Consumer brauchen keine Anpassung — Default-Verhalten ist identisch zu v2.0.1.

Outline-Doku ebenfalls aktualisiert:

Test plan

  • Consumer-Repo mit Tests (z. B. Butterfly) auf diesen Branch pinnen, PR-Build muss grün laufen und test-report-Job muss weiterhin Tests aggregieren.
  • Aonyx-bom (oder anderes BOM-Repo) mit run-tests: false pinnen, PR-Build muss grün laufen, test-report-Job darf nicht erscheinen, keine test-reports-*-Artefakte werden hochgeladen.
  • release-please macht aus diesem feat:-Commit einen Minor-Bump → v2.1.0.

Generated by Claude Code

claude added 3 commits May 13, 2026 08:39
Add a `run-tests` boolean input (default `true`) so the same workflow can
be reused for projects with tests, projects without tests, and BOM-only
publications. When `run-tests` is false the JUnit upload step and the
aggregation job are skipped, and the resolved gradle task falls back from
`build test` to `build`. Existing consumers keep their current behavior.
…lint

markdownlint flags MD012 (multiple consecutive blank lines) on every
release-please-generated CHANGELOG.md entry. Add a markdownlint-cli2
config that ignores the file — release-please owns the formatting.

lychee gets a 403 from Cloudflare on docs.renovatebot.com even though
the page resolves fine in browsers. Add the host to .lycheeignore so
the link check stops failing on a healthy URL.
lychee 0.23 removed --exclude-mail (mail isn't checked unless
--include-mail is passed). The lychee-action v2 ships lychee 0.23.0
since the last bump, so the default lychee-args were rejected as
"unexpected argument". Drop the flag — behavior is unchanged.
@TheMeinerLP TheMeinerLP merged commit 3508e38 into main May 13, 2026
4 checks passed
@TheMeinerLP TheMeinerLP deleted the claude/fix-gradle-test-config-JT1yW branch May 13, 2026 08:50
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