server icon indicating copy to clipboard operation
server copied to clipboard

Syncing only releases

Open jdcrensh opened this issue 8 years ago • 4 comments

Backstroke is great for syncing commits from a branch. But in certain situations, it'd be ideal to create backstroke PRs only for upstream releases.

Taking create-react-app as an example, I don't really want to have Backstroke create a PR for every other commit on master, because the changes may not be ready for production. I'd want to have a PR created whenever a release is cut.

What would be the best approach here?

  • Option to sync releases instead of choosing a branch, then use the API to find the latest release during sync.
  • Be less tied to what Github deems as a release, instead allowing configuration of an optional commit message regex, eg. ^Publish.*$. This would be easier to implement but less user friendly and prone to false positives.

jdcrensh avatar Oct 24 '17 18:10 jdcrensh

@jdcrensh Cool idea!

Once requirement is that Github requires pull requests to be made off of a branch, which would make pull requests off of tags a bit hard. I'm exploring https://github.com/backstrokeapp/server/issues/74, which could potentially make syncing to a tag possible.

A commit message regex would definitely be easier, though an inferior solution.

Is this a feature you'd like to work on? This isn't something that I would likely work on in anytime soon.

Thanks for the idea!

1egoman avatar Oct 24 '17 21:10 1egoman

I hadn't considered that limitation of PRs, thanks for pointing it out before I spent too much time on it. May have some time to explore it in a week or so, but I'll watch #74 cause it might be a prerequisite.

jdcrensh avatar Oct 24 '17 23:10 jdcrensh

@jdcrensh Here's a PR for #74: https://github.com/backstrokeapp/worker/pull/2

1egoman avatar Oct 31 '17 11:10 1egoman

@jdcrensh That PR I previously mentioned has been merged. Don't know if this is still something you're interested in working on but if so the backend code for that feature is in a good place.

1egoman avatar Dec 11 '17 22:12 1egoman