ansible/awx

Remove some of the eslint-disable comments

Open

#11.707 aperta il 9 feb 2022

Vedi su GitHub
 (4 commenti) (0 reazioni) (0 assegnatari)Python (3333 fork)batch import
Hacktoberfestcomponent:uigood first issuetype:enhancementtype:task

Metriche repository

Star
 (13.071 star)
Metriche merge PR
 (Merge medio 24g 6h) (30 PR mergiate in 30 g)

Descrizione

ISSUE TYPE
  • Feature Idea
SUMMARY

There are several places in the code basis that we use - eslint-disable and certain rule, we need to verify which ones we should keep in place, and make changes accordingly.

This PR https://github.com/ansible/awx/pull/11568 added a couple more in order to speed up the react-scripts bump process.

Guida contributor

Direzione di ricerca
Ispeziona ogni commento eslint disable nel codice, verifica se la violazione della regola è ancora presente e rimuovi il commento se la violazione è stata risolta o la regola non è più necessaria.
Tech stack
javascriptreact
Dominio
frontend
Tipo issue
Refactoring
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
2
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
1-3 ore
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
JavaScripteslint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
75