On Tue, 28 May 2024 21:37:43 GMT, Andy Goryachev <ango...@openjdk.org> wrote:

>> modules/javafx.graphics/src/test/java/test/com/sun/javafx/css/FixedCapacitySetTest.java
>>  line 1:
>> 
>>> 1: package test.com.sun.javafx.css;
>> 
>> The Copyright is missing from this class and the `if` and `for` clauses have 
>> no space after.
>> I think we usually do that, still minor of course. Tests are looking good to 
>> me!
>
> oops I missed that!  I think we must include the standard copyright header 
> per OCA agreement.

I always forget the header, thanks, I added it!

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1316#discussion_r1617920591

Reply via email to