theman550 / DAT257-K9

Project for course DAT257
1 stars 1 forks source link

Update ESLint with React compatibility and rules #22

Closed AndreasWJ closed 4 years ago

AndreasWJ commented 4 years ago

La till React som ESLint plugin. Ändrade även lite i konfigurationen för att likna den gamle .eslintrc.js. Uppdaterade .js filer till .jsx för det är vad ESLint förväntar sig.

ITJohan commented 4 years ago

Jag trodde vi var överens om att använda airbnbs konfiguration? Du accepterade ju även https://github.com/theman550/DAT257-K9/pull/14. Det jag kan hålla med om är att ändra filnamn till .jsx eftersom airbnb kör på det, men 4 spaces har jag personligen väldigt svårt för och airbnb rekommenderar 2 spaces: https://github.com/airbnb/javascript#whitespace. Kan även gå med på att vi kan ta bort kravet på proptypes eftersom vi antagligen inte kommer ha så många komponenter som vi kommer dela. Men det hade inte skadat heller och är en bra vana att ha i React.

AndreasWJ commented 4 years ago

Jag trodde vi var överens om att använda airbnbs konfiguration? Du accepterade ju även #14. Det jag kan hålla med om är att ändra filnamn till .jsx eftersom airbnb kör på det, men 4 spaces har jag personligen väldigt svårt för och airbnb rekommenderar 2 spaces: https://github.com/airbnb/javascript#whitespace. Kan även gå med på att vi kan ta bort kravet på proptypes eftersom vi antagligen inte kommer ha så många komponenter som vi kommer dela. Men det hade inte skadat heller och är en bra vana att ha i React.

Yes vi kör på det. Skrev 4 spaces eftersom har väldigt svårt att läsa 2 spaces i kod själv, och det var det jag specificerade i den originella eslintrc.js.

Proptypes är lite anti-javascript(dynamisk typing), men har inget emot det heller. Tar la 30 sekunder att skriva en sådan.

Vi kan alltid lägga till eller ta bort regler. Airbnb är bara en bas som man brukar utgå från.