Uh oh!
There was an error while loading. Please reload this page.
ci: Setup gitflow process - #6890
Conversation
size-limit report 📦
|
Lms24
left a comment
There was a problem hiding this comment.
Seems like a good workflow to me!
Also, great diagram! Once we all agree on this, let's make sure to document this flow in our docs directory and maybe link to it from CONTRIBUTING or similar?
0507cec to
929e5a3Comparemydea
commented
Jan 30, 2023
Added docs here: https://github.com/getsentry/sentry-javascript/blob/fn/gitflow/CONTRIBUTING.md Note: I took this opportunity and extracted the publishing stuff out as well. |
Lms24
left a comment
There was a problem hiding this comment.
LGTM. I like the automatic approach master/develop syncing approach and I think going forward, moving to gitflow will solve our release blocking/multiple teams working on the repo problem. Thanks for updating documentation, I think it makes a lot of sense now.
Uh oh!
There was an error while loading. Please reload this page.
mydea
commented
Jan 31, 2023
FYI I updated this slightly:
|
This PR (WIP) sets up some Gitflow processes.
Some requirements that need to be done in order for this to work/make sense:
mastertodevelop(so have them for both branches)developthe default branch in GithubHow it should work (overview)
developbranch. Any PRs are merged into developHow to do a release going forward?
Potential streamlining/improvement?
One way in which this could be improved, is to auto-trigger the release whenever we merge into master.
In order to do this, we'll need to somehow know at this point:
We may be able to parse this somehow, but I'd say this is an area for future improvement, if we decide so.
What's missing?
We'll probably have to tweak caching etc. settings a bit to streamline this - e.g. what needs to run when merging develop-> master, what needs to run when merging master->develop, etc.