-
Notifications
You must be signed in to change notification settings - Fork 25
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
Add a Github Actions script that synchronizes the template every month. #56
Conversation
Can you show your test branch results? Once you proved this works, the second part of the task is to update all of the recently updated repositories with the script as well. |
Doesn't this directly merge in the content instead of opening a pull-request? I think it would be less error prone to open a PR against the repo instead of directly merging the changes, so we can double check before they get in the repo, which is also in the spec. |
Okay, the sync.yml script now creates a pull request. Here is a pull request on the "testing" branch cloned from "test/cd": https://github.com/zyrafal/ts-template/pull/1 And on the "testing-2" branch I merged the pull request: https://github.com/zyrafal/ts-template/tree/testing-2 However, I haven't yet managed to get the other script to work, I'm still working on it. |
I didn't manage to get the script that adds it to every repo working as a Github Action, but it works when you run it in shell:
Also, I created a new repository with the template and successfuly ran the sync script there (though there were no changes that needed to by synced): https://github.com/zyrafal/test-repo/pull/1 |
.github/workflows/sync.yml
Outdated
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we rename this to sync-template.yml
for clarity?
Okay, I think it works now. |
The workflow seemed to work: Meniole/command-start-stop#11 It just doesn't seem super easy to use because I would have many files to delete, since the template covers a lot of cases. I Wonder if we could add some configuration values to ignore certain files (the README for example) dynamically. |
Co-authored-by: Mentlegen <9807008+gentlementlegen@users.noreply.github.com>
Can you get it to auth as our bot like in our other ci |
Done. |
This requires every repo to have our credentials stored because this comes from an automated workflow and doesn't get the credentials from the kernel, I don't think that's very practical. |
Every repo has our credentials stored. It's an org secret. |
Before accepting this pull and paying out the reward, please open pulls to the other active repositories as per the spec. Ensure that they link to the issue so we can track all of them. Thank you! |
Resolves #54
The script is located in .github/workflows/sync.yml and it synchronizes the template at 0:00 UTC the 1st day of every month.
I cloned a branch called "test/cd", which was 128 commits behind the "development" branch, into a new branch called "testing", and then I successfuly ran the script on it, proving that it works.