Skip to content

#2127 split url updater build run#2141

Open
Hiepiscus wants to merge 4 commits into
devonfw:mainfrom
Hiepiscus:feature/2127-split-url-updater-build-run
Open

#2127 split url updater build run#2141
Hiepiscus wants to merge 4 commits into
devonfw:mainfrom
Hiepiscus:feature/2127-split-url-updater-build-run

Conversation

@Hiepiscus

Copy link
Copy Markdown

This PR fixes #2127

Implemented changes:

  • update-urls.yml: Split the existing Build and run url updater workflow step into two separate steps

Testing instructions

  1. Trigger the workflow containing the url-updater job.
  2. Verify that the workflow now contains two separate steps:
    • Build url updater
    • Run url updater
  3. Confirm that the Maven build and test logs are shown in the Build url updater step.
  4. Confirm that the url-updater execution logs are shown in the Run url updater step.
  5. Verify that the workflow completes successfully and behaves the same as before.

Notes

The workflow could not be fully executed in my fork because it requires
repository secrets that are not available in fork repositories.

Checklist for this PR

Make sure everything is checked before merging this PR. For further info please also see
our DoD.

  • When running mvn clean test locally all tests pass and build is successful
  • PR title is of the form #«issue-id»: «brief summary» (e.g. #921: fixed setup.bat). If no issue ID exists, title only.
  • 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 or there is no issue (might happen for very small PRs)
  • 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"

Checklist for tool commandlets

Not applicable — this PR does not add a new tool commandlet.

@CLAassistant

CLAassistant commented Jul 9, 2026

Copy link
Copy Markdown

CLA assistant check
All committers have signed the CLA.

@Hiepiscus Hiepiscus changed the title Feature/2127 split url updater build run #2127 split url updater build run Jul 9, 2026
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 29021687063

Coverage decreased (-0.05%) to 72.18%

Details

  • Coverage decreased (-0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 3 coverage regressions across 2 files.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

3 previously-covered lines in 2 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/version/VersionSegment.java 2 90.03%
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java 1 78.33%

Coverage Stats

Coverage Status
Relevant Lines: 16708
Covered Lines: 12569
Line Coverage: 75.23%
Relevant Branches: 7458
Covered Branches: 4874
Branch Coverage: 65.35%
Branches in Coverage %: Yes
Coverage Strength: 3.18 hits per line

💛 - Coveralls

@Hiepiscus Hiepiscus added the workflow GitHub actions (CI,CD,update urls/CVEs) label Jul 10, 2026
@hohwille hohwille moved this from 🆕 New to Team Review in IDEasy board Jul 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow GitHub actions (CI,CD,update urls/CVEs)

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Split "build and run url updater" step into separate build and run steps

5 participants