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
When I was looking for info about how to structure my PR, I checked out the "A typical contribution" section of the Guide to Rustc development book, because based on the available sections, that seemed the most likely. I wouldn't have thought to look in the Coding Conventions sections (I wouldn't consider a PR a piece of code is why I'm guessing)
As an addition to this, while it might not be strictly necessary, or even within the scope of the dev-guide, perhaps it could be helpful to include a brief comment (or even a quick overview of the required commands) about why the "No merge commits" point is made.
For new contributors like myself, this wasn't immediately clear to me, and I'm sure it's the same for others.
perhaps it could be helpful to include a brief comment (or even a quick overview of the required commands) about why the "No merge commits" point is made.
Activity
alex-griffiths commentedon Aug 19, 2020
As an addition to this, while it might not be strictly necessary, or even within the scope of the dev-guide, perhaps it could be helpful to include a brief comment (or even a quick overview of the required commands) about why the "No merge commits" point is made.
For new contributors like myself, this wasn't immediately clear to me, and I'm sure it's the same for others.
jyn514 commentedon Sep 30, 2020
@alex-griffiths this was added as part of #890: https://rustc-dev-guide.rust-lang.org/git.html#no-merge-policy