open-duelyst/duelyst

[P1] Enable linting for ESLint override files

Open

Aperta il 10 set 2022

Vedi su GitHub
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (3443 star) (526 fork)batch import
backendenhancementfrontendhelp wanted

Descrizione

We have several files with overrides set in .eslintrc.json:

    {
      "files": ["app/**/*.js"],
      "rules": {"camelcase": ["warn"]}
    },
    {
      "files": ["test/**/*.js"],
      "rules": {"no-unused-expressions": ["off"]}
    }

We should remove these overrides and resolve the underlying linting issues.

Guida contributor

Tech stack
javascriptnodejs
Dominio
tooling
Tipo issue
refactor
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 hours
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
stale
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
clear
Prerequisiti
JavaScriptESLintnpm
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
85
Direzione di ricerca
The issue requires modifying the .eslintrc.json file to remove the 'files' overrides for app/**/*.js and test/**/*.js, then running ESLint and fixing any resulting linting errors in those files. The changes are localized and well defined.