Skip to content

#2051: fix release workflow#2157

Open
quando632 wants to merge 2 commits into
devonfw:mainfrom
quando632:feature/2051-release-workflow-fix
Open

#2051: fix release workflow#2157
quando632 wants to merge 2 commits into
devonfw:mainfrom
quando632:feature/2051-release-workflow-fix

Conversation

@quando632

@quando632 quando632 commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

This PR fixes #2051

Implemented changes:

  • Changed the workflow-level permissions in .github/workflows/release.yml from contents: read to contents: write.

Testing instructions

This behavior only exists inside a GitHub Actions runner and needs no org secrets to verify (GITHUB_TOKEN is auto-provided):

Note for maintainers: confirm the devonfw org setting Settings → Actions → General → "Workflow permissions" allows workflows to request write; otherwise this line alone is not sufficient.


Checklist for this PR

  • When running mvn clean test locally all tests pass and build is successful (N/A — CI workflow YAML change only, no code)
  • PR title is of the form #«issue-id»: «brief summary»
  • PR top-level comment summarizes what has been done and contains link to addressed issue(s)
  • PR and issue(s) have suitable labels
  • Issue is set to In Progress and assigned to you
  • You followed all coding conventions
  • You have added the issue implemented by your PR in CHANGELOG.adoc unless issue is labeled with internal
  • You have formulated clear instructions on how to test your contribution under "Testing instructions"

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Jul 14, 2026
@quando632 quando632 added SCM software-configuration-management (github actions, internal processes, git or github utilization) workflow GitHub actions (CI,CD,update urls/CVEs) labels Jul 14, 2026
@quando632 quando632 self-assigned this Jul 14, 2026
@coveralls

coveralls commented Jul 14, 2026

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 29327329969

Warning

No base build found for commit a88c41e on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 72.199%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 16733
Covered Lines: 12589
Line Coverage: 75.23%
Relevant Branches: 7464
Covered Branches: 4881
Branch Coverage: 65.39%
Branches in Coverage %: Yes
Coverage Strength: 3.18 hits per line

💛 - Coveralls

@quando632 quando632 marked this pull request as ready for review July 14, 2026 11:06
@quando632 quando632 moved this from 🆕 New to Team Review in IDEasy board Jul 14, 2026
@quando632 quando632 moved this from Team Review to 👀 In review in IDEasy board Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

SCM software-configuration-management (github actions, internal processes, git or github utilization) workflow GitHub actions (CI,CD,update urls/CVEs)

Projects

Status: 👀 In review

Development

Successfully merging this pull request may close these issues.

Release workflow failing publishing release metadata on github

2 participants