[Rev 01] RFR: Add checkout instructions to PR body
Jorn Vernee
jvernee at openjdk.java.net
Tue Feb 18 10:44:18 UTC 2020
On Tue, 18 Feb 2020 07:54:51 GMT, Erik Helin <ehelin at openjdk.org> wrote:
>> 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);`
@edvbld Thanks Erik,
That test is being skipped when running on Windows and I'm seeing a lot of "STANDARD_ERROR"s in the build log for the workflow, so not sure if this was a false negative.
I gave the latest version a run on WSL. There are a lot of failing tests if I do that, unrelated to this PR, but at least the particular test is now passing.
-------------
PR: https://git.openjdk.java.net/skara/pull/434
More information about the skara-dev
mailing list