Skip to content

#1940: Create SpyderUrlUpdater#1944

Open
tineff96 wants to merge 1 commit into
devonfw:mainfrom
tineff96:feature/1940-spyder-urlupdater
Open

#1940: Create SpyderUrlUpdater#1944
tineff96 wants to merge 1 commit into
devonfw:mainfrom
tineff96:feature/1940-spyder-urlupdater

Conversation

@tineff96
Copy link
Copy Markdown

@tineff96 tineff96 commented May 13, 2026

This PR fixes #1940

Implemented changes:

  • SpyderUrlUpdater
  • updated filter logic for getCustomVersionFilter. Returning string-array instead of string to allow multiple custom filters.

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 13, 2026
@tineff96 tineff96 moved this from 🆕 New to Team Review in IDEasy board May 13, 2026
@tineff96 tineff96 self-assigned this May 13, 2026
@tineff96 tineff96 marked this pull request as ready for review May 13, 2026 14:51
@tineff96 tineff96 changed the title #1940: implemented spyder url-updater, test, changes for filter logic #1940: Create SpyderUrlUpdater May 13, 2026
@tineff96 tineff96 added enhancement New feature or request urls ide-urls repo and related processes and features internal Nothing to be added to CHANGELOG, only internal story labels May 13, 2026
@coveralls
Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 25806717174

Coverage decreased (-0.02%) to 70.662%

Details

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

Uncovered Changes

No uncovered changes found.

Coverage Regressions

16 previously-covered lines in 3 files lost coverage.

File Lines Losing Coverage Coverage
com/devonfw/tools/ide/url/updater/UpdateManager.java 13 0.0%
com/devonfw/tools/ide/version/VersionSegment.java 2 89.24%
com/devonfw/tools/ide/url/updater/AbstractUrlUpdater.java 1 82.7%

Coverage Stats

Coverage Status
Relevant Lines: 15492
Covered Lines: 11411
Line Coverage: 73.66%
Relevant Branches: 6926
Covered Branches: 4430
Branch Coverage: 63.96%
Branches in Coverage %: Yes
Coverage Strength: 3.12 hits per line

💛 - Coveralls

@AdemZarrouki AdemZarrouki self-assigned this May 15, 2026
@AdemZarrouki AdemZarrouki self-requested a review May 15, 2026 11:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request internal Nothing to be added to CHANGELOG, only internal story urls ide-urls repo and related processes and features

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Create SpyderUrlUpdater

3 participants