#1651: push and commit status.json files to ide-urls-status after running the URL update worklfow#2003
Conversation
…s to the new repo ide-urls-status
Coverage Report for CI Build 26944006085Coverage decreased (-0.02%) to 71.03%Details
Uncovered ChangesNo uncovered changes found. Coverage Regressions3 previously-covered lines in 2 files lost coverage.
Coverage Stats💛 - Coveralls |
tineff96
left a comment
There was a problem hiding this comment.
I tested the workflow end-to-end in a fork.
Test setup:
Created a test branch (test/2003-test-branch) in my fork:
tineff96/IDEasy
tineff96/ide-urls
tineff96/ide-urls-status
Adapted the workflow in tineff96/IDEasy to run against the test branch (test/2003-test-branch)
Executed the workflow based on the provided template
(reference run: https://github.com/AdemZarrouki/IDEasy/actions/runs/26902445623)
Result:
The migration of status.json files from ide-urls to ide-urls-status worked as expected.
The workflow correctly commits:
status.jsonfiles are committed toide-urls-status- all other files are committed to
ide-urls
No unexpected files were committed
Important notes for reviewers / future testers:
Setting up the test environment is slightly non-trivial because it requires:
Forks of both repositories (ide-urls and ide-urls-status)
Adjusting the workflow to use the test branch
You must define the following secrets for commits to work:
ACTION_PUSH_TOKEN
BUILD_USER
BUILD_USER_EMAIL
Without BUILD_USER and BUILD_USER_EMAIL, the commit step will fail.
Conclusion:
From my perspective, the implementation behaves correctly and is ready for review.
This PR fixes #1651
Implemented changes:
update-urls.ymlto commit and pushstatus.jsonfiles to repoide-urls-statusstatus.jsonshould not be pushed toide-urlsACTION_PUSH_TOKENso that it have access to the new create repoide-urls-status. This need to be done by @hohwille as he has access to that functionality.status.jsonfiles fromide-urls.Testing instructions
Please add conscise, understandable instructions on how a reviewer can test/verify the functionality of your contribution here:
update-urls.ymlto include your test branch. A workflow example was created and it successfully updates bothide-urlsandide-urls-status. https://github.com/AdemZarrouki/IDEasy/actions/runs/26902445623ACTION_PUSH_TOKENand 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.
mvn clean testlocally all tests pass and build is successful#«issue-id»: «brief summary»(e.g.#921: fixed setup.bat). If no issue ID exists, title only.In Progressand assigned to you or there is no issue (might happen for very small PRs)with
internal