Skip to content
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

Checkstyle: add more rules, commit IntelliJ config #377

Merged
merged 4 commits into from
Sep 12, 2023
Merged

Conversation

ljacqu
Copy link
Member

@ljacqu ljacqu commented Sep 12, 2023

No description provided.

- This commit re-orders all checks to be aligned with the groups on the Checkstyle website. No rules were changed or added.
- This is an initial draft for a Checkstyle configuration I want to use in all projects: some fine-tuning will still be necessary to not only suit ConfigMe. This commit does not fix any of the new issues (which are esp. Javadoc and magic numbers)
…style plugin config

- Fine-tune some of the checks to be more generally applicable
- Fix violations except in code that is planned to be changed soon
- Commit IntelliJ's checkstyle-idea.xml plugin config so that the right file and Checkstyle version are used out of the box
@coveralls
Copy link

Coverage Status

coverage: 99.392%. remained the same when pulling 20941f3 on checkstyle-revamp into bb74744 on master.

@ljacqu ljacqu merged commit 20941f3 into master Sep 12, 2023
5 checks passed
@ljacqu ljacqu deleted the checkstyle-revamp branch September 12, 2023 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants