Skip to content

#1910: create mvnd url updater#1921

Open
MarvMa wants to merge 3 commits into
devonfw:mainfrom
MarvMa:feature/#1910-create-mvndUrlUpdater
Open

#1910: create mvnd url updater#1921
MarvMa wants to merge 3 commits into
devonfw:mainfrom
MarvMa:feature/#1910-create-mvndUrlUpdater

Conversation

@MarvMa
Copy link
Copy Markdown
Contributor

@MarvMa MarvMa commented May 12, 2026

This PR fixes #1910

Implemented changes:

  • Added MvndUrlUpdater class with filtering
  • Added Tests

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

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board May 12, 2026
@MarvMa MarvMa self-assigned this May 12, 2026
@MarvMa MarvMa added enhancement New feature or request urls ide-urls repo and related processes and features mvn related to apache maven build tool labels May 12, 2026
@MarvMa MarvMa moved this from 🆕 New to Team Review in IDEasy board May 12, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

coveralls commented May 12, 2026

Coverage Report for CI Build 25734421628

Coverage increased (+0.05%) to 70.672%

Details

  • Coverage increased (+0.05%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • 13 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

13 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/url/updater/UpdateManager.java 13 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 15497
Covered Lines: 11416
Line Coverage: 73.67%
Relevant Branches: 6932
Covered Branches: 4435
Branch Coverage: 63.98%
Branches in Coverage %: Yes
Coverage Strength: 3.12 hits per line

💛 - Coveralls

Copy link
Copy Markdown
Contributor

@satorus satorus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, tested it on WSL and the urls are updated in my local ide-urls repo. Nice work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request mvn related to apache maven build tool urls ide-urls repo and related processes and features

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Create MvndUrlUpdater

3 participants