search

watchdogpolska / small_eod

System służący do usprawnienia obiegu dokumentów Stowarzyszenia na potrzeby prowadzonych litygacji Stowarzyszenia
http://small-eod.vercel.app
MIT License
56 stars 43 forks source link

Zaktualizowanie lub zmiana narzędzi do automatycznego formatowania kodu #341

Open kuskoman opened 4 years ago

kuskoman commented 4 years ago

Narzędzia dostarczone z frameworkiem Ant Design Pro w niektórych przypadkach wymagają użycia menadżera pakietów npm, podczas gdy projekt docelowo rozwiązuje wszystkie zależności za pomocą yarna.

Również ilość skryptów przeznaczonych do formatowania kodu w package.json budzi we mnie zakłopotanie:

"lint": "npm run lint:js && npm run lint:style && npm run lint:prettier",
    "lint-staged": "lint-staged",
    "lint-staged:js": "eslint --ext .js,.jsx,.ts,.tsx ",
    "lint:fix": "eslint --fix --cache --ext .js,.jsx,.ts,.tsx --format=pretty ./src && npm run lint:style",
    "lint:js": "eslint --cache --ext .js,.jsx,.ts,.tsx --format=pretty ./src",
    "lint:prettier": "check-prettier lint",
    "lint:style": "stylelint --fix \"src/**/*.less\" --syntax less",
    "prettier": "prettier -c --write \"**/*\"",

Kolejnym problemem jest użycie nieaktualnych narzędzi, które nie są już aktualizowane ( https://github.com/JamesHenry/typescript-estree )

Myślę, że rozwiązaniem byłaby zmiana narzędzi/konfiguracji przeznaczonych formatowania tak, aby "wyrządzić jak najmniej szkód" w postaci zmian w kodzie utrudniających odczytanie historii danego pliku w gicie, które mogłyby wyniknąć ze zmian w regułach formatowania

Dodaję referencję do #338, ponieważ ten issue blokuje wykonanie tamtego

kuskoman commented 4 years ago

Informacyjnie: Spędziłem na tym issue ponad 40 minut jakiś czas temu wciąż mając problemy ze znalezieniem podobnej konfiguracji i działającej dobrze z TypeScriptem (mówię głownie o ESLint)

Myślę, że powinniśmy przemyśleć do czego tak właściwie chcemy używać linterów i skonfigurować je pod to

@magul

magul commented 4 years ago

Ja jestem na ta, jak wrócisz z kubernetowej przygody to przegadamy temat na którymś z codwutygodniowych spotkań.

kuskoman commented 4 years ago

@magul już jestem dostępny od poniedziałku, po prostu mi się nie chciało nic robić :P jestem za tym, aby to omówić i zdecydować na to jak chcemy używać linterów, etc.

mik-laj commented 4 years ago

Ta zmiana jest powiązaana https://github.com/watchdogpolska/small_eod/pull/445 Ja potrzebowałem dodać dodatkowe sprawdzenia, które poprawia końce linii i inne. Moje IDE automatycznie wprowadza zmiany w plikach, gdy robię CTRL+S.