RFR: Section on PRs [v2]

[Jesper Wilhelmsson]

On Thu, 7 Apr 2022 18:05:31 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Jesper Wilhelmsson has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added sentence about keeping PR up to date
>
> src/index.md line 1460:
> 
>> 1458: ## Rebase before creating the PR
>> 1459: 
>> 1460: It's likely that other people have pushed changes to the code base since you created your branch. Make sure to pull the latest changes and rebase your fix on top of that before creating your PR. This is a courtesy issue. Your reviewers shouldn't have to read your patch on top of old code that has since changed. This is hopefully obvious in cases where the upstream code has gone through cleanups or refactorings, and your patch may need similar cleanups in order to even compile. But even in cases where only smaller changes has been done, the reviewers shouldn't have to react to issues like "that line of code was moved last week, why is it back there?".
> 
> Minor: "in cases where only smaller changes has been done" --> "...have been done"

Fixed.

> src/index.md line 1466:
> 
>> 1464: ~~~
>> 1465: 
>> 1466: After the PR has been published, rebasing, force-pushing, and similar actions is strongly discouraged. Such actions will disrupt the workflow for reviewers who fetch the PR branch. Pushing new changes is fine (and even merging if necessary) for a PR under review. Incremental diffs and other tools will help your reviewers see what you have changed. In the end, all commits will be squashed into a single commit automatically, so there're actually no drawbacks what so ever to making several commits to a PR branch during review.
> 
> Minor:
> "rebasing, force-pushing, and similar actions is strongly discouraged" --> "...are strongly discouraged"
> 
> Also, "whatsoever" is one word.

Fixed.

-------------

PR: https://git.openjdk.java.net/guide/pull/79