Open zepumph opened 9 months ago
It looks like it would need to be patched in a few places to avoid this type of thing.
Is there a quick and dirty spot where we could fail out just so that we know it isn't a supported feature?
Or better yet, could it just become a ternary when it comes to figuring out the branch name (untested made up code):
get branchName( repo ) { return this.repo === repo? this.branch : '${repo}-${this.branch}'; }
I recently was tired and wrote this maintenance code (as part of a larger MR patch in https://github.com/phetsims/build-a-nucleus/issues/215). It broke things!
It ended up creating a build-a-nucleus-1.1 branch. That makes me sad, and seems like an easy thing to improve upon, but I couldn't really crack it when looking at the code. @jonathanolson could you help me brainstorm even a spot for an assertion like "You can't patch the branch of a release branch that you have as a needed patch".