Enigma icon indicating copy to clipboard operation
Enigma copied to clipboard

Project formatting

Open 2xsaiko opened this issue 5 years ago • 3 comments

The code should either be formatted with tabs or with spaces. Right now there's a mix of both.

2xsaiko avatar Jun 08 '20 23:06 2xsaiko

I would suggest applying the same if not similar checksyle that is used across the other fabric projects.

modmuss50 avatar Jun 08 '20 23:06 modmuss50

Eeeeh, even having to fix IDEA indenting the file wrongly in some of the classes you edit is way less of a pain than having to deal with checkstyle imo. I'd really rather not.

Unless there's an IDE plugin to auto format the file according to the config and not having to go through the gradle task.

2xsaiko avatar Jun 08 '20 23:06 2xsaiko

The gradle task only validates it. There is an idea plugint that will validate it as you type.

As for a formatter there is an idea config somewhere that will do most if not all of it for you. I usually just do it by hand tho.

Failing that adding a editor config file to specify tabs and passing an auto formatter over it would be a good improvement

modmuss50 avatar Jun 09 '20 00:06 modmuss50

Fixed as of 9c736848fb7aa82d295b3aa2946e6cd132ee998f

NebelNidas avatar Sep 21 '22 21:09 NebelNidas