You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We need documentation to guide developers in resolving this common problem. 2 devs start with the same base commit, the first one to create a PR is okay, but the 2nd dev to create a PR will have some conflicts in their PR, even though it runs fine.
Action Items
write doc to explain and guide to find the solution
check the website team's guide on the same topic
if there are common sections
copy the relevant portion into a new guide in the Engineering CoP docs repo. Note which projects are referencing the guide.
link from our docs while removing those sections
notify website team to examine the content and link it from their docs
This might be something we want the engineering cop to write. If they won't write is, we will write our own custom version.
fyliu
changed the title
Add documentation on how to resolve PR merge conflicts when another contributor made a previous PR
[docs] Document how to resolve PR merge conflicts when another contributor made a previous PR
Jun 3, 2024
fyliu
changed the title
[docs] Document how to resolve PR merge conflicts when another contributor made a previous PR
[docs] How to resolve PR merge conflicts when another contributor made a previous PR
Jun 3, 2024
fyliu
changed the title
[docs] How to resolve PR merge conflicts when another contributor made a previous PR
docs: How to resolve PR merge conflicts when another contributor made a previous PR
Jun 10, 2024
Dependency
Overview
We need documentation to guide developers in resolving this common problem. 2 devs start with the same base commit, the first one to create a PR is okay, but the 2nd dev to create a PR will have some conflicts in their PR, even though it runs fine.
Action Items
Resources/Instructions
Discussion
The preferred solution is to rebase the new PR to the new HEAD of the main branch and resolve any conflicts so that the PR can be merged smoothly.
See discussion on this topic
The text was updated successfully, but these errors were encountered: