diffplug / diffplug/spotless

Add support for npm-groovy-lint

Offen
#1,121 1 Kommentar 2 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen
enhancement
Vorherrschende Sprache
Java
Sterne
5.6k
Forks
559
Ø Merge
1 T. 14 Std.
Gemergte PRs (30 T.)
43

Beschreibung

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!

Beitragsleitfaden

Beitragsleitfaden öffnen

Rechercherichtung

Beginne mit dem Lesen von Issue #246 und den vorhandenen Spotless-Integrationen, um herauszufinden, wie externe Linter konfiguriert und aufgerufen werden. Prüfe, wie Spotless npm-groovy-lint über Node.js und den separaten Java-Prozess aufrufen würde; abgeschlossen ist die Arbeit, wenn der Linter über die Konfigurationsschicht der Bibliothek mit angemessener Abdeckung unterstützt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
groovy, java, node.js
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.