sindresorhus/eslint-plugin-unicorn

Rule proposal: Forbid reassign top level variable in functions

Chiusa

#1092 aperta il 30 gen 2021

Vedi su GitHubGuida contributor
 (6 commenti) (1 reazione) (0 assegnatari)JavaScript (468 fork)user submission
help wantednew rule

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 1g 16h) (399 PR mergiate in 30 g)

Descrizione

It's dangerous, and maybe get unexpected result when refactoring. In following example, if we change foo to async, or inside foo somehow reenter foo, cache may got change.

Fail

let cache; 

function foo() {
	cache = {};
	// ...
}

export default foo;

Pass

function foo() {
	let cache;

	// ...
}

Guida contributor

Direzione di ricerca
Esamina le implementazioni delle regole ESLint esistenti nel repository eslint plugin unicorn, in particolare le regole semplici che vietano pattern specifici. Utilizza ESLint RuleTester per i test. La regola dovrebbe impedire la riassegnazione di variabili let o var dichiarate al livello superiore di un modulo, solo quando all'interno di una funzione. Osserva i selettori AST per individuare AssignmentExpression all'interno di FunctionDeclaration o ArrowFunctionExpression che assegnano a una variabile dichiarata nello scope Program. Includi test per i casi di fallimento e successo come delineato nell'issue.
Tech stack
javascript
Dominio
developer experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
ESLintJavaScript
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
65