Skip to content

Add timeout-minutes and concurrency groups to all workflows - #185

Merged
AndreasIgel merged 2 commits into
java-helpers:mainfrom
AndreasIgel:feature/163-workflow-timeouts-concurrency
Jul 19, 2026
Merged

AndreasIgel merged 2 commits into
java-helpers:mainfrom
AndreasIgel:feature/163-workflow-timeouts-concurrency

Conversation

@AndreasIgel

Copy link
Copy Markdown
Collaborator

Upstream counterpart of fork PR AndreasIgel#10. Head branch: AndreasIgel/simple-builders-fork:feature/163-workflow-timeouts-concurrency → base java-helpers/simple-builders:main.

Summary

Adds the two resource guards missing from every workflow.

Fixes #163

  • timeout-minutes on every job so a hung step can't run to the 6-hour default: maven.yml build 20, codeql.yml analyze 120, dependency-review.yml 10, maven-central-release.yml 30.
  • concurrency group on every workflow:
    • CI/scan (maven, codeql, dependency-review) → group keyed on github.ref with cancel-in-progress: true (cancel superseded runs for the same ref).
    • maven-central-release → fixed group maven-central-release with cancel-in-progress: false, so releases serialize and an in-progress release is never cancelled mid-deploy (avoids a partial publish).

Validation

  • actionlint passes for all four workflows.
  • Workflow-only change; no Maven impact.

Existing red checks (build, dependency-review) are the fork's missing SONAR_TOKEN/CODECOV_TOKEN and disabled Dependency graph — unrelated to this change.

Every job now has a timeout-minutes (build 20, CodeQL 120, dependency
review 10, release 30) so a hung step can't run to the 6h default. Every
workflow has a concurrency group: CI/scan workflows cancel superseded
runs for the same ref; the release workflow serializes and never cancels
an in-progress release to avoid partial publishes.

Refs java-helpers#163

Co-Authored-By: Andreas Igel <andreas.igel@computacenter.com>
@codecov

codecov Bot commented Jul 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@AndreasIgel
AndreasIgel merged commit bef6860 into java-helpers:main Jul 19, 2026
6 checks passed
@devin-ai-integration
devin-ai-integration Bot deleted the feature/163-workflow-timeouts-concurrency branch August 16, 2026 21:11
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.

Add timeout-minutes and concurrency groups to all workflows

1 participant