-
Notifications
You must be signed in to change notification settings - Fork 221
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
build: Added .editorconfig to project #571
Conversation
@AlmasB should this be PR be marked as |
@abhinayagarwal yeah looks like a |
I added TABs in 2 files: one Also, is there a way to make the maven build fail if the check fails? |
Well, the Maven Plugin is not really working as expected. Hence my proposal is to remove it. However, I'll contact the author and will ask for help there. Nevertheless, we can keep the For now, I've removed the Maven plugin. If we want to reformat, we could still apply checkstyle. Checkstyle works. |
This reverts commit 77acf7c.
Yes, checkstyle is a good option. Let's take it up as a separate issue. |
Issue
Fixes #485
Adds a very basic
.editorconfig
to the project and the related Maven plug-in to automatically format files on demand.README.md has been updated accordingly on how to make use of EditorConfig and where to get the plug-in's for the IDEs.
Progress
Remark 2024-03-19:
The Maven plugin has been removed as the setup did actually not work as expected. However, the
.editorconfig
works well in IntelliJ IDEA and Eclipse (when equipped with the plugin).