-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix template sync #222
Fix template sync #222
Conversation
This reverts commit 5f091ed.
Co-authored-by: Gregor Sturm <[email protected]>
* Add retry with backoff also for clone * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
A PR has been generated to the instance repo: You can check out the PR to preview your changes |
- better logging - check out release tag instead of main - put repo owner in PR branch name to deal with forks
This allows to re-run the entire action on failures without creating noise in the repos where the update was successful.
@flying-sheep, do you have any clue why the test is failing? Locally all tests pass... |
The "real" sync run was successful 🎉. A couple of repos failed, but the script recovered gracefully. I set the respective repos to skip in scverse/ecosystem-packages@10c060c. |
awesome! Once the script tests are fixed properly, we can really call this done (well, actually expanding the tests wouldn’t hurt either) |
@flying-sheep, could you please take a look at the tests? I don't understand why they are failing and can't reproduce the failure locally. |
Close #208