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

Formater fix #83

Open
wants to merge 18 commits into
base: main
Choose a base branch
from
Open

Formater fix #83

wants to merge 18 commits into from

Conversation

nomorehumor
Copy link
Collaborator

@nomorehumor nomorehumor commented Jun 30, 2023

The PR contains changes in formatter configurations, as well as some of the formatted files

The usage of formatter is as follows (if it doesn't work by default):

  1. In Intellij go to Preferences > Editor > Code Style > Java, and with the cog near 'Scheme' dropdown choose the checkstyle config from repo (checkstyle_checks.xml)
  2. For files with errors, you can either right-click on file in Project Explorer > Reformat Code or fix it manually (only option for some errors)
  3. (optional) If you want, you can also add Checkstyle plugin locally and import config there

The formatter doesn't check for JavaDocs and package naming (for now)

@nomorehumor nomorehumor changed the title added dry-run Formater fix Jun 30, 2023
@nomorehumor
Copy link
Collaborator Author

Other files should be formatted in other branches to avoid numerous conflicts

@nomorehumor nomorehumor mentioned this pull request Jul 24, 2023
@Ralfons-06
Copy link
Collaborator

The PR contains changes in formatter configurations, as well as some of the formatted files

The usage of formatter is as follows (if it doesn't work by default):

1. In Intellij go to Preferences > Editor > Code Style > Java, and with the cog near 'Scheme' dropdown choose the checkstyle config from repo (checkstyle_checks.xml)

2. For files with errors, you can either right-click on file in Project Explorer > Reformat Code or fix it manually (only option for some errors)

3. (optional) If you want, you can also add Checkstyle plugin locally and import config there

The formatter doesn't check for JavaDocs and package naming (for now)

In IntelliJ the Checkstyle Plugin has to be installed beforehand otherwise, it's not possible to choose the check_style.xml in the editor settings

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants