On Fri, 7 Apr 2023 22:40:32 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Thiago Milczarek Sayao has updated the pull request incrementally with two 
>> additional commits since the last revision:
>> 
>>  - Revert "Make intellij see :systemTests dependecies"
>>    
>>    This reverts commit dca7eab24958e1214147b7d291f0faf52ea27ddf.
>>  - Make intellij see :systemTests dependecies
>
> .gitignore line 67:
> 
>> 65: 
>> 66: #mac folder
>> 67: .DS_Store
> 
> These seem unrelated to this PR (and in the case of `hs_err`, there is a 
> typo).

Removed them

> checkstyle.xml line 1:
> 
>> 1: <?xml version="1.0"?>
> 
> I'm not keen on adding yet another top-level file in our repo. Can it go 
> somewhere else?

I've looked for a place, but didn't found one that would make sense. It could 
be used on eclipse as well and probably on netbeans.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1009#discussion_r1167667502
PR Review Comment: https://git.openjdk.org/jfx/pull/1009#discussion_r1167667672

Reply via email to