RFR: JDK-8223373: Remove IntelliJ IDEA specific files from the source code repository

Thiago Milczarek Sayao tsayao at openjdk.org
Sun Jan 29 19:41:04 UTC 2023


On Mon, 23 Jan 2023 23:55:51 GMT, Thiago Milczarek Sayao <tsayao at openjdk.org> wrote:

> This PR does:
> 
> - Remove specific Idea files and let it be imported from gradle;
> - Adds checkstyle (to use with checkstyle plugin - it will let you know style mistakes);
> - Configures auto-format to sun style (with the changes mentioned in [Code Style Rules](https://wiki.openjdk.org/display/OpenJFX/Code+Style+Rules));
> - Automatically sets Copyright notice (updates year too);
> - Run configurations for samples/toys and builds.

I would appreciate feedback from intellij idea users.

CheckStyle plugin is also available for [Eclipse](https://checkstyle.org/eclipse-cs/#!/) if anyone is willing to set it up.

It caught many simple mistakes on the code.

@andy-goryachev-oracle You might like it :)

I would like to add manual tests to gradle so it would be imported by the IDE in a way it doesn't build (most are single java files that can be run directly) but code completion works. Does anyone know how to do that?

I still cannot run manual tests directly from the IDE. I think it's close to working, but something is missing and I'm not a gradle ninja.

Eclipse outputs the same error when running manual tests, so I think it's not possible to fix without restructuring the folder. 

Ready for review.

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

PR: https://git.openjdk.org/jfx/pull/1009


More information about the openjfx-dev mailing list