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

[CODE-STYLE] Tooling für Clean-Code und Code-Style Konfigurieren #1226

Closed
AMatutat opened this issue Nov 20, 2023 · 1 comment
Closed

[CODE-STYLE] Tooling für Clean-Code und Code-Style Konfigurieren #1226

AMatutat opened this issue Nov 20, 2023 · 1 comment

Comments

@AMatutat
Copy link
Contributor

AMatutat commented Nov 20, 2023

Wir haben eine grundlegende Konfiguration für Spotless, Spotbugs und Checkstyle, wobei wir (bis auf Spotless) diese weitgehend ignorieren. Einige Einstellungen sind teilweise nicht optimal für unseren Anwendungsfall geeignet (beispielsweise hat ein Framework mehr Methoden pro Klasse, um bessere API-Zugriffe zu ermöglichen).

Es wäre sinnvoll, sich jetzt die Zeit zu nehmen und geeignete Parameter für diese Tools zu überlegen. Möglicherweise gibt es auch bessere oder zusätzliche Tools, die wir in Betracht ziehen können.


Edit: Checkstyle ist für immer verbannt (#1316 und #1318). Die kriegen ihre eigene Konfiguration nicht in den Griff (vgl. checkstyle/checkstyle#14123 und checkstyle/checkstyle#14211). Davon abgesehen: Wer möchte im Jahr 2024 immer noch unübersichtliche XML-Konfigurationsfiles manuell editieren?!

Edit: SpotBugs ist ganz nett, um typische Anti-Pattern zu finden bzw. gar nicht erst einzubauen. Allerdings wurde während der gesamten aktiven Projektentwicklung nicht darauf geachtet, und wer soll das jetzt nachträglich alles gerade ziehen?! #1320 entfernt SpotBugs aus der Konfiguration.

Edit: Damit bleibt nur noch Spotless für die Formatierung übrig. Bzgl. der Einstellungen (Google-Style vs. AOSP-Style) ist #944 offen.

@cagix
Copy link
Member

cagix commented Jan 3, 2024

hier gibts nicht mehr zu tun.

@cagix cagix closed this as not planned Won't fix, can't repro, duplicate, stale Jan 3, 2024
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