-
-
Notifications
You must be signed in to change notification settings - Fork 701
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
Merge 4.x.x branch into master #772
Comments
@meeber @lucasfcosta @keithamus Thanks |
No objections. Good luck and god speed, brave @meeber |
I think a rebase would be cleaner, but I'm not sure it is the ideal way of getting those two branches together. I feel like a traditional merge would be optimal choice here, given the big commit difference. However, I'm not very experienced with it, cause I've never done such a big merge, it would be good to have a third opinion about this. Anyway, good luck with that merge and have a good time on your plane 😄 |
Still on vacay. Forgot to fetch the 4.x.x branch before my flight here and, unlike last time I flew, I couldn't get even 5 minutes of Internet during the flight. Will better prepare for the flight back this weekend and try to knock this out! |
First and foremost @meeber - enjoy your vacation! Here's a handy to-do for you to keep up with this:
|
@keithamus @lucasfcosta I'm headed to the opposite side of the world on Thursday and am gonna be stuck in airplanes for a ridiculous amount of time. That means I need a big project to keep me occupied :D
I'd like to take the opportunity to merge the 4.x.x branch into master in preparation for the canary release. Any objections? If no objections, any suggestions? I've never merged two branches together with this substantial of a commit difference (250ish and counting). Would it better to rebase the 4.x.x branch using the master branch? Or perform a traditional merge? Or some other approach? I realize that either way there's gonna be a conflict resolution stage, but I'm a bit fuzzy on the github flow for this kind of operation.
Thanks :D
The text was updated successfully, but these errors were encountered: