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

Merge 4.x.x branch into master #772

Closed
meeber opened this issue Aug 15, 2016 · 5 comments · Fixed by #783
Closed

Merge 4.x.x branch into master #772

meeber opened this issue Aug 15, 2016 · 5 comments · Fixed by #783
Milestone

Comments

@meeber
Copy link
Contributor

meeber commented Aug 15, 2016

@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

@vieiralucas
Copy link
Member

@meeber @lucasfcosta @keithamus
Just want to remember you guys to not release 4.0.0 before #691 is closed.

Thanks

@keithamus
Copy link
Member

No objections. Good luck and god speed, brave @meeber

@lucasfcosta
Copy link
Member

lucasfcosta commented Aug 16, 2016

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 😄
Let me know if you need anything.

@meeber
Copy link
Contributor Author

meeber commented Aug 30, 2016

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!

@keithamus
Copy link
Member

First and foremost @meeber - enjoy your vacation! Here's a handy to-do for you to keep up with this:

  • enjoy vacation
  • spend quality time
  • everything else important to you personally
  • work on merging 4.x branch

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 a pull request may close this issue.

4 participants