Skip to content

Fix linter npm-groovy-lint#5073

Closed
Jorenar wants to merge 1 commit intodense-analysis:masterfrom
Jorenar:fix-npmgroovylint
Closed

Fix linter npm-groovy-lint#5073
Jorenar wants to merge 1 commit intodense-analysis:masterfrom
Jorenar:fix-npmgroovylint

Conversation

@Jorenar
Copy link
Contributor

@Jorenar Jorenar commented Dec 4, 2025

I wanted to use npm-groovy-lint to check some Gradle files, but I was not getting any diagnostics.

Changes:

  • removed filename field from handler's output (it was pointing to temporary file),
  • removed --loglevel warning from default options (so by default we see linter's defaults),
  • added --noserver as a default option (to avoid lingering process after closing Vim),
  • appending rule name to diagnostic message (so user knows which rules to toggle/modify in linter's config),
  • adjusted tests,
  • updated docs.

@Jorenar Jorenar closed this Dec 5, 2025
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.

1 participant