add eslint config

[yoper12]

rules to be agreed, these are available:

• https://eslint.org/docs/latest/rules/
• https://eslint.style/packages/js#rules

[banocean]

Tak właściwie co chcemy sprawdzać tymi lintami?

[yoper12]

jeszcze nie wiem co ani czy w ogóle, ale możesz sobie zobaczyć jak to wygląda aktualnie tu i tu

[yoper12]

no ale generalnie chcemy sprawdzać jakość kodu i żeby kod był w miarę w jednym stylu

[banocean]

@yoper12 deno run --allow-env --allow-read --allow-sys --allow-write npm:web-ext -- lint 🤔, i nie potrzeba package.json tak przy okazji

[yoper12]

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

[yoper12]

zresztą nawet gdybyśmy eslint używali przez deno, to nadal słabo z integracją

[banocean]

chyba nie ma co dodawać, widać czy przechodzi jak spojrzysz na ostatni commit

[banocean]

to chyba ustalone już było, że plików od konkretnych IDE nie dajemy

[banocean]

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

[yoper12]

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

no nawet jeśli, to z tego co widzę web-ext nie sprawdza prawie niczego poza manifestem. poza tym fajnie gdyby błędy były widoczne jeszcze przed spushowaniem, a mało komu będzie się chciało specjalnie instalować deno i za każdym razem uruchamiać tę długą komendę.

[banocean]

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

no nawet jeśli, to z tego co widzę web-ext nie sprawdza prawie niczego poza manifestem. poza tym fajnie gdyby błędy były widoczne jeszcze przed spushowaniem, a mało komu będzie się chciało specjalnie instalować deno i za każdym razem uruchamiać tę długą komendę.

Można zawsze zainstalować web-ext normalnie 🤷‍♂️

[yoper12]

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

no nawet jeśli, to z tego co widzę web-ext nie sprawdza prawie niczego poza manifestem. poza tym fajnie gdyby błędy były widoczne jeszcze przed spushowaniem, a mało komu będzie się chciało specjalnie instalować deno i za każdym razem uruchamiać tę długą komendę.

Można zawsze zainstalować web-ext normalnie 🤷‍♂️

no kiedy mówię że to coś nic nie sprawdza

[yoper12]

+ weź się najpierw zajmij #120

[wombatus2137]

+ weź się najpierw zajmij #120

Żeby można w końcu było #2

[banocean]

• weź się najpierw zajmij Add settings #120

Żeby można w końcu było #2

Tam dalej brakuje implementacji dla zmian spow9dowanyvh przez część patchy i dla uonetplus.vulcan.net.pl i dziennik-logowanie.vulcan.net.pl

[banocean]

• weź się najpierw zajmij Add settings #120

Żeby można w końcu było #2

Tam dalej brakuje implementacji dla zmian spow9dowanyvh przez część patchy i dla uonetplus.vulcan.net.pl i dziennik-logowanie.vulcan.net.pl

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

no nawet jeśli, to z tego co widzę web-ext nie sprawdza prawie niczego poza manifestem. poza tym fajnie gdyby błędy były widoczne jeszcze przed spushowaniem, a mało komu będzie się chciało specjalnie instalować deno i za każdym razem uruchamiać tę długą komendę.

Można zawsze zainstalować web-ext normalnie 🤷‍♂️

no kiedy mówię że to coś nic nie sprawdza

Działa bez package.json, serio nie wiem w czym problem

[yoper12]

• weź się najpierw zajmij Add settings #120

Żeby można w końcu było #2

Tam dalej brakuje implementacji dla zmian spow9dowanyvh przez część patchy i dla uonetplus.vulcan.net.pl i dziennik-logowanie.vulcan.net.pl

wiem że deno fajne, ale ten web-ext to chyba nie ma tak dobrej integracji z edytorami jak eslint

ale my chyba chcemy dodać linting PR'ów i commitów, a to jak ktoś ma u siebie edytor skonfigurowany to trochę nie mój problem

no nawet jeśli, to z tego co widzę web-ext nie sprawdza prawie niczego poza manifestem. poza tym fajnie gdyby błędy były widoczne jeszcze przed spushowaniem, a mało komu będzie się chciało specjalnie instalować deno i za każdym razem uruchamiać tę długą komendę.

Można zawsze zainstalować web-ext normalnie 🤷‍♂️

no kiedy mówię że to coś nic nie sprawdza

Działa bez package.json, serio nie wiem w czym problem

no w tym, że nic nie sprawdza, a linter raczej powinien sprawdzać. porównaj sobie wynik z web-ext do tego z eslinta

[yoper12]

• weź się najpierw zajmij Add settings #120

Żeby można w końcu było #2

Tam dalej brakuje implementacji dla zmian spow9dowanyvh przez część patchy i dla uonetplus.vulcan.net.pl i dziennik-logowanie.vulcan.net.pl

no jak, przecież wszystko jest. jedynie co to fajnie gdyby niebieskie rzeczy na efebie były czerwone

[yoper12]

a no dobra, logowania też nie ma, ale to nie zmienia faktu że możesz się tym zająć i dodać te rzeczy

[banocean]

a no dobra, logowania też nie ma, ale to nie zmienia faktu że możesz się tym zająć i dodać te rzeczy

Jakbym miał na to wszystko czas to bym nie potrzebował was do czegokolwiek, po za tym code review może r9bic więcej osób niż ja