Kevin Rushforth
Kevin Rushforth
I also see no reason to make this change. In addition to the points mentioned above, gradle 9 will require JDK 17 as a minimum, so you won't even be...
Ultimately, this will be up to @abhinayagarwal or @johanvos to decide whether to accept this PR. I don't see any real benefit in doing this, but maybe they will.
This should be discussed on the mailing list first. I recommend moving it to Draft for now. If it goes forward it will need a CSR. /reviewers 2 reviewers /csr
@Ziad-Mid Can you be the second reviewer? /reviewers 2
Yes, I always disable the gradle daemon in my `$HOME/.gradle/gradle.properties`. Our CI builds and test runs also do this (our Jenkins scripts pass `--no-daemon` to all invocations of gradle). We've...
I'm reasonably sure there was a good reason for the code in NGNode doing what it did. This will need very careful review and testing before we would accept it....
Reviewers: @arapte @andy-goryachev-oracle @Maran23 wait for either @arapte or me to review the proposed Prism changes in this PR.
Reviewers: @kevinrushforth @lukostyra /reviewers 2
@arapte Would you also be able to look into this in case I am not able to?
> One thing I could not do is to enumerate the properties getter via reflection or BeanInfo/PropertyDescriptor because the class is not accessible com.sun.javafx.application.preferences.PlatformPreferences Did try to use `pref.getClass()` for...