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

[FEATURE] Add linting #124

Closed
msbarry opened this issue Mar 8, 2022 · 2 comments
Closed

[FEATURE] Add linting #124

msbarry opened this issue Mar 8, 2022 · 2 comments
Labels
internal Improve the experience for contributors

Comments

@msbarry
Copy link
Contributor

msbarry commented Mar 8, 2022

To improve consistency between people who may use different editors, we should adopt a java linter like checkstyle or sonarlint. You should be able to run it from the command line, through plugins to popular editors, and a github action should highlight any issues as comments on a PR.
@msbarry msbarry added the internal Improve the experience for contributors label Mar 8, 2022
@msbarry
Copy link
Contributor Author

msbarry commented Apr 13, 2022 •
edited
Loading

Created #178 from this to discuss some options

@msbarry msbarry mentioned this issue Apr 13, 2022
Merged
@msbarry
Copy link
Contributor Author

msbarry commented Apr 24, 2022

Closed by #177

@msbarry msbarry closed this as completed Apr 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
internal Improve the experience for contributors
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@msbarry