sindresorhus/eslint-plugin-unicorn

Rule proposal: `consistent-conditional-object-spread`

Chiusa

#2363 aperta il 21 mag 2024

Vedi su GitHubGuida contributor
 (6 commenti) (3 reazioni) (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

Description

When spreading an object conditionally, we have two style: ? : and &&. I prefer user consistently to use &&.

Pass

const name = Math.random() > 0.5 'foo' : undefined;
const student = {age: 12, ...(name&&{name})};

Fail

const name = Math.random() > 0.5 'foo' : undefined;
const student = {age: 12, ...(name ? {name} : {})};

Proposed rule name

consistent-conditional-object-spread

Additional Info

No response

Guida contributor

Direzione di ricerca
Inizia leggendo la guida ai contributi di ESLint plugin unicorn (https://github.com/sindresorhus/eslint plugin unicorn/blob/main/.github/CONTRIBUTING.md). Guarda le implementazioni delle regole esistenti come 'consistent destructuring' come riferimento. La regola dovrebbe rilevare gli spread condizionali che usano l'operatore ternario vs && e imporre uno stile. Gli esempi 'pass' e 'fail' nell'issue definiscono il comportamento atteso. Nessuna PR collegata o assegnatario, quindi questo è aperto per i contributi.
Tech stack
javascripttypescript
Dominio
tooling
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.
Nuova
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
ESLint rule developmentAST understandingJavaScript/TypeScript basics
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50