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

Re-enable ESLint rules #1209

Open
2 tasks done
Miguel7373 opened this issue Nov 25, 2024 · 1 comment · Fixed by #1260
Open
2 tasks done

Re-enable ESLint rules #1209

Miguel7373 opened this issue Nov 25, 2024 · 1 comment · Fixed by #1260
Assignees

Comments

@Miguel7373
Copy link
Collaborator

Miguel7373 commented Nov 25, 2024

Im Ticket #1135 wurde ESLint eingeführt. Nun hat es im Winter ein paar rules, die noch aufgeschaltet wurden, da sie nicht das formatting betreffen. Die müssen jetzt gelöst werden.
Gehe dafür auf einen seperaten branch ausgehend von feature/1135/formatter und deaktiviere zuerst das stylistic plugin. kommentiere danach alle regeln aus, so das nur noch die .ts Regeln ausgeführt werden.
Führe danach npm run lint:format aus. Commite diesen Stand, dies wird als grundlage dienen.
Optimalerweise sollte npm run lint nun keine Fehler mehr ausweisen.
Aktiviere nun nacheinander die deaktivierten Reglen und löse die damit verbundenen Probleme.

Orientiere dich dafür an den Kommentaren

Im eslint.config.mjs sind die regeln.

To rebase the branch rebase onto the 1135/formatter branch (or onto main if the branch is already merged)
for

Anforderungen

  • Es wurden alle rules von ToDO: bis zu den stylistic rules entfernt

Akzeptanzkriterien

  • Die rules unter dem ToDO existieren nicht mehr.
  • Der formatter läuft durch.
@nevio18324 nevio18324 self-assigned this Dec 16, 2024
@nevio18324
Copy link
Collaborator

nevio18324 commented Dec 16, 2024

Todo's

  • Stylistic plugin deaktivieren
  • npm run lint:format ausführen dannach den state pushen
  • (Optional falls nicht zu viel Zeit benötigt) Soll npm run lint keine errors melden
  • Dannach von oben nach unten die Regeln aktivieren und die beschriebenen Probleme lösen

Bereits Aktivierte Regeln

  • ban-ts-comment
  • no-unused-vars
  • no-unused-imports
  • no-undef
  • no-namespace
  • prefer-rest-params
  • no-empty-functions
  • lines-around-comment
  • no-empty-lifecyle-methods
  • component-class-suffix

@nevio18324 nevio18324 linked a pull request Dec 17, 2024 that will close this issue
@nevio18324 nevio18324 removed a link to a pull request Dec 24, 2024
@nevio18324 nevio18324 linked a pull request Dec 24, 2024 that will close this issue
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 a pull request may close this issue.

2 participants