Skip to content

#1651: push and commit status.json files to ide-urls-status after running the URL update worklfow#2003

Open
AdemZarrouki wants to merge 2 commits into
devonfw:mainfrom
AdemZarrouki:feature/1651-migrate-status.json-files
Open

#1651: push and commit status.json files to ide-urls-status after running the URL update worklfow#2003
AdemZarrouki wants to merge 2 commits into
devonfw:mainfrom
AdemZarrouki:feature/1651-migrate-status.json-files

Conversation

@AdemZarrouki

@AdemZarrouki AdemZarrouki commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

This PR fixes #1651

Implemented changes:

  • update the workflow in update-urls.yml to commit and push status.json files to repo ide-urls-status
  • status.json should not be pushed to ide-urls
  • The migration of status.json is alreday done in #1651: migrate status.json files to ide-urls-status ide-urls-status#1
  • Before Merging we need to update ACTION_PUSH_TOKEN so that it have access to the new create repo ide-urls-status. This need to be done by @hohwille as he has access to that functionality.
  • After merging this we need to create another PR to delete all status.json files from ide-urls.

Testing instructions

Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:

  1. This can be tested in a test branch but for that you need to adapt update-urls.yml to include your test branch. A workflow example was created and it successfully updates both ide-urls and ide-urls-status. https://github.com/AdemZarrouki/IDEasy/actions/runs/26902445623
  2. Also you need to include a personal ACTION_PUSH_TOKEN and activate the URL Update workflow in your fork.

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"

@github-project-automation github-project-automation Bot moved this to 🆕 New in IDEasy board Jun 4, 2026
@AdemZarrouki AdemZarrouki added urls ide-urls repo and related processes and features workflow GitHub actions (CI,CD,update urls/CVEs) labels Jun 4, 2026
@coveralls

Copy link
Copy Markdown
Collaborator

Coverage Report for CI Build 26944006085

Coverage decreased (-0.02%) to 71.03%

Details

  • Coverage decreased (-0.02%) 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 89.24%
com/devonfw/tools/ide/tool/ide/IdeToolCommandlet.java 1 78.33%

Coverage Stats

Coverage Status
Relevant Lines: 15823
Covered Lines: 11729
Line Coverage: 74.13%
Relevant Branches: 7066
Covered Branches: 4529
Branch Coverage: 64.1%
Branches in Coverage %: Yes
Coverage Strength: 3.14 hits per line

💛 - Coveralls

@AdemZarrouki AdemZarrouki marked this pull request as ready for review June 4, 2026 09:50
@AdemZarrouki AdemZarrouki self-assigned this Jun 4, 2026
@AdemZarrouki AdemZarrouki moved this from 🆕 New to Team Review in IDEasy board Jun 4, 2026
@AdemZarrouki AdemZarrouki requested a review from tineff96 June 5, 2026 11:21
tineff96 added a commit to tineff96/IDEasy that referenced this pull request Jun 5, 2026
tineff96 added a commit to tineff96/IDEasy that referenced this pull request Jun 5, 2026
tineff96 added a commit to tineff96/IDEasy that referenced this pull request Jun 8, 2026
tineff96 added a commit to tineff96/IDEasy that referenced this pull request Jun 8, 2026
tineff96 added a commit to tineff96/IDEasy that referenced this pull request Jun 8, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

urls ide-urls repo and related processes and features workflow GitHub actions (CI,CD,update urls/CVEs)

Projects

Status: Team Review

Development

Successfully merging this pull request may close these issues.

Move status.json files out of ide-urls to own repo

3 participants