Skip to content
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

Break each replacement out of the update-content file and provide a system for writing more replacements #29

Merged
merged 36 commits into from
Sep 16, 2020

Conversation

tonygaetani
Copy link
Contributor

We've used this repo at 6 River Systems to migrate our default branches to main which has been great! Thank you so much @mheap for doing the heavy lifting here with this script!

We added a replacement for circleci, if there is no circleci file in the repo then nothing happens.

We also changed how replacements happen, and made an extensible system for adding new replacements.

I've also done a bunch of work writing bash scripts to automatically update local git repos on development machines, that has been done in our internal tooling but if there is interest I can share some of the code and the lessons I've learned here as well

@mheap mheap changed the base branch from main to 6RiverSystems-enhancements September 16, 2020 10:00
@mheap
Copy link
Owner

mheap commented Sep 16, 2020

Hey @tonygaetani, thanks for this, and sorry it's taken so long to be reviewed.

There are a few small changes I'd like to make as there are changes in this PR (such as removing the default old value) that need reverting. However, I'm aware that this is coming from your main branch. With that in mind, I'll merge this PR in to another branch, make the changes and tag a release with circleci support later today

@mheap mheap merged commit b501e08 into mheap:6RiverSystems-enhancements Sep 16, 2020
mheap pushed a commit that referenced this pull request Sep 16, 2020
…ystem for writing more replacements (#29)

Co-authored-by: Harrison Muskat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants