RFR: Add checkout instructions to PR body

Jorn Vernee jvernee at openjdk.java.net
Mon Feb 17 11:16:21 UTC 2020


On Thu, 13 Feb 2020 14:26:29 GMT, Erik Helin <ehelin at openjdk.org> wrote:

>> Hi,
>> 
>> I'd like let the pr bot add the fetch comamnd found in the automatic emails to the PR body, so that people using only GitHub can see the instructions as well.
>> 
>> This adds a new "Instructions" section to the body of the PR message:
>> 
>>> ## Instructions
>>> To checkout these changes locally:
>>> ```bash
>>> $ git fetch https://git.openjdk.java.net/panama-foreign pull/16/head:pull/16
>>> $ git checkout pull/16
>>> ```
>> 
>> Testing:
>> - Running `.\gradlew test`
>> 
>> Any other recommendations for testing this?
>> 
>> Thanks,
>> Jorn
> 
> Thanks Jorn, good idea �� I think we might want to overhaul the design a little bit to accommodate for this new section. I added in some more sections that might appear in a pull request -  what do you think about the following?
> 
>> -----------------------
>> ### Progress
>> - [x] Change must not contain extraneous whitespace
>> - [ ] Change must be properly reviewed (1/2)
>>
>> ### Issues
>> * [JDK-8712393](localhost): Add download instructions to pull request
>>
>> ### Contributors
>> * Robin Westberg ([rwestberg](localhost))
>> 
>> ### Reviewers
>> * Erik Duveblad ([ehelin](localhost) - **Reviewer**)
>> * Joe Darcy ([darcy](locahost) - **Reviewer**) �� Re-review required (review applies to 7de15f5c44a6f4f5a18afda090d52b7b69c7d09c)
>>
>> ### Download
>> `$ git fetch https://git.openjdk.java.net/panama-foreign pull/16/head:pull/16`
>> `$ git checkout pull/16`

Hey Erik,

I like the new format that you suggest as well :) I've applied the suggestions.

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

PR: https://git.openjdk.java.net/skara/pull/434


More information about the skara-dev mailing list