Skip to content

Conversation

@SDallmann
Copy link
Contributor

@SDallmann SDallmann commented Jan 28, 2026

This PR fixes #1691

Implemented changes:

Updated references to the UrlUpdaters in the documentation IDEasy-contribution-getting-started.adoc:

e.g. from cli/src/main/java/com/devonfw/tools/ide/url/updater/JsonUrlUpdater.java
to url-updater/src/main/java/com/devonfw/tools/ide/url/updater/JsonUrlUpdater.java


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 Jan 28, 2026
@SDallmann SDallmann changed the title #1691 updated references in documentation #1691 updated references to UrlUpdaters in the documentation Jan 28, 2026
@coveralls
Copy link
Collaborator

Pull Request Test Coverage Report for Build 21445417289

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 70.446%

Totals Coverage Status
Change from base Build 21350416345: 0.0%
Covered Lines: 10482
Relevant Lines: 14293

💛 - Coveralls

@hohwille hohwille moved this from 🆕 New to 👀 In review in IDEasy board Jan 29, 2026
@hohwille hohwille added this to the release:2026.02.001 milestone Jan 29, 2026
Copy link
Member

@hohwille hohwille left a comment

Choose a reason for hiding this comment

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

@SDallmann thanks for your PR and fixing all these broken links. All correct now.👍
Ready for merge.

@hohwille hohwille merged commit 815d34c into devonfw:main Jan 29, 2026
4 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in IDEasy board Jan 29, 2026
@hohwille hohwille added documentation Improvements or additions to documentation enhancement New feature or request labels Jan 29, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation enhancement New feature or request

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Update references to UrlUpdaters in the documentation

3 participants