RFR: 1589: Improve TestIssue and TestPullRequest [v2]
Erik Joelsson
erikj at openjdk.org
Fri Sep 9 17:03:13 UTC 2022
On Fri, 9 Sep 2022 15:20:14 GMT, Magnus Ihse Bursie <ihse at openjdk.org> wrote:
> This is quite a massive PR. It took some time to look thought. I lost track of some changes but I think I got to the bottom of most of these. I can definitely see a drawback with using the "var" syntax when reviewing diffs on Github -- when I'm in the IDE I can get tooltips telling me about the type, but now I had to jump back and forth to look for definitions to understand how the code tied together.
>
> So in the end, I think it looks good. The fact that it is only test changes makes it definitely less risky.
Thank you for taking the time!
-------------
PR: https://git.openjdk.org/skara/pull/1370
More information about the skara-dev
mailing list