[Rev 01] RFR: Add checkout instructions to PR body

Erik Helin ehelin at openjdk.java.net
Tue Feb 18 07:57:14 UTC 2020


On Mon, 17 Feb 2020 14:59:26 GMT, Erik Helin <ehelin at openjdk.org> wrote:

>> The pull request has been updated with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase.
> 
> Nice work! Just a small nit, you can use a Markdown [thematic break](https://spec.commonmark.org/0.29/#thematic-break) instead of using the `<hr />` HTML tag directly.

It seems like the test `org.openjdk.skara.bots.pr.ContributorTests > prBodyUpdates` is failing on both Linux and Mac. I think you have to change these two lines:

- https://github.com/openjdk/skara/blob/master/bots/pr/src/test/java/org/openjdk/skara/bots/pr/ContributorTests.java#L335
   `                if (line.equals("## Contributors")) {`
- https://github.com/openjdk/skara/blob/master/bots/pr/src/test/java/org/openjdk/skara/bots/pr/ContributorTests.java#L355
   `                assertNotEquals("## Contributors", line);`

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

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


More information about the skara-dev mailing list