RFR: 282: Make the review requirements more explicit in the progress list [v3]

[Guoxiong Li]

On Fri, 29 Apr 2022 22:18:07 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

>>> I would probably have put the logic for generating this string right here in the getter. 
>> 
>> Fixed. Please note the field `reviewRequirements` is not `final` now.
>> 
>>> I realize this gets a lot of testing through the ReviewersTest above, but it would make sense to have cheap and quick tests of just this functionality here in the jcheck package.
>> 
>> Added tests.
>
> That's fine, thanks!
> 
> I now realize that you put the space and parentheses in the string here. I think this method should just return the requirements and let the visitor add the surrounding formatting.

Fixed.

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

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