CommandHelper icon indicating copy to clipboard operation
CommandHelper copied to clipboard

Add vscode presetting and entity toggle swim event

Open Anatoliy057 opened this issue 7 months ago • 3 comments

I added presets for vscode, specifically the code formatting configuration. Tried to make it the best fit for the project, but there are inconsistencies in it project itself, and some of the rules just can't be turned off. Anyway this is only for vscode and only if you execute formating manually.

Anatoliy057 avatar Jun 11 '25 14:06 Anatoliy057

The project setup is described in the Maven pom file. Specific IDE setups should be generated from that, this is why there are no IntelliJ/Eclipse/NetBeans/VSC configuration files in this git. To apply the CheckStyle configuration of the project in VSC, there seems to be an extension that you can install: https://marketplace.visualstudio.com/items?itemName=shengchen.vscode-checkstyle. Why do you want to add these VSC-specific files?

Pieter12345 avatar Jun 11 '25 15:06 Pieter12345

@Pieter12345 these files prompt vscode for a list of recommended extensions, one of which you mentioned and the settings for them (literally a couple of things and file paths).

the only thing questionable is the config file formatting, but added just as a why not

Anatoliy057 avatar Jun 11 '25 16:06 Anatoliy057

I definitely would like that configuration to be derived directly from the source of truth (in this case, the checkstyle file), rather than having an IDE specific configuration that has to be maintained. I do like the extension suggestion. Would be nice to enforce that some way better (if you use VSC, you MUST have this extension) but obviously that's bad for various reasons. In any case, the maintenance burden on this shouldn't be too high, and I do like batteries included things, so maybe it's fine to keep this. We don't change the code formatting standard often (ever), so once it's written, that should be it, unless the IDE itself changes the format of the file or something. In any case, I reserve the right to delete that config file later, but I think it's fine for now.

LadyCailin avatar Jun 18 '25 14:06 LadyCailin