RFR: More on testing

[David Holmes]

On Mon, 20 Feb 2023 19:01:54 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

>> * Added clarifications and expectations around testing and review
>> * Moved GHA section (unchanged)
>> * Removed outdated stuff from the glossary
>
> src/guide/hotspot-development.md line 5:
> 
>> 3: See [Working With Pull Requests](#working-with-pull-requests) for generic guidance and requirements around pushing changes. For the HotSpot codebase there are a few additional requirements:
>> 4: 
>> 5: * Your change must have been approved by two Committers out of which at least one is also a Reviewer
> 
> The original requirement, from the Oracle-internal wiki ("Hotspot Code Review Guidelines"), was "At least two reviewers (one of which is a Reviewer)".  The "two Committers" requirement suggested here (taken from the openjdk wiki) was a quiet change introduced when the openjdk wiki page was created based on the internal wiki.  In earlier (private) discussion this change was questioned.  We want to encourage thoughtful and knowledgeable reviews even from folks who are new or only lightly connected to the project and so don't have Committer status (yet), rather than effectively saying such reviews don't count for anything in the official process.

I agree - please keep this as "two reviewers".

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

PR: https://git.openjdk.org/guide/pull/98