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.

Sign up for GitHub

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

Jump to bottom

Add codestyle checks #45

Merged
merged 3 commits into from
Nov 29, 2021
Merged

Add codestyle checks #45

merged 3 commits into from
Nov 29, 2021

Conversation

justusbunsi
Copy link
Member

As title. Identifies:

  • misuse of tab character
  • missing newline at end of file
  • inconsistent line separators
  • trailing spaces on lines
  • unused imports
  • misuse of * imports

@justusbunsi
Integrate maven-checkstyle-plugin to build
d92ecc0 
Signed-off-by: justusbunsi <61625851+justusbunsi@users.noreply.github.com>
@justusbunsi
Apply codestyle rules
600f866 
Signed-off-by: justusbunsi <61625851+justusbunsi@users.noreply.github.com>
@justusbunsi
Ignore files generated during build
73798b7 
Signed-off-by: justusbunsi <61625851+justusbunsi@users.noreply.github.com>
@lafriks lafriks merged commit c9335dd into jenkinsci:master Nov 29, 2021
@justusbunsi justusbunsi deleted the codestyle-checks branch November 29, 2021 08:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lafriks lafriks lafriks approved these changes

Assignees
No one assigned
Labels
chore
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@justusbunsi @lafriks