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

Add support for npm-groovy-lint #1121

Open
Alex-Weatherhead opened this issue Feb 6, 2022 · 1 comment
Open

Add support for npm-groovy-lint #1121

Alex-Weatherhead opened this issue Feb 6, 2022 · 1 comment

Comments

@Alex-Weatherhead
Copy link

I would like to add support for the npm-groovy-lint linter as per #246.

There is still the ugliness of having Spotless call Node.js only to then call a separate Java process, but I think that the sugary configuration layer provided by this library is valuable enough. Moreover, I don't think it's worth re-implementing the logic in Java, especially as the linter is under active development (and thus, whoever ported the logic would be forced to always play catch-up or lose out on the new features being provided).

I'm happy to take this work on if there are no objections!

@nedtwigg
Copy link
Member

nedtwigg commented Feb 9, 2022

Very happy to take such a PR! It might be easier for you if you first take a look at one of the following issues.

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

No branches or pull requests

2 participants