On Tue, 28 May 2024 21:31:53 GMT, Marius Hanl <mh...@openjdk.org> wrote:
>> John Hendrikx has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Added 100% coverage test for FixedCapacitySet > > 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. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1316#discussion_r1617917861
