sindresorhus/eslint-plugin-unicorn

Rule proposal: consistent class member order

Chiusa

#1110 aperta il 18 feb 2021

Vedi su GitHubGuida contributor
 (5 commenti) (2 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

Enforce statements in this order

Originally posted by @fisker in https://github.com/sindresorhus/eslint-plugin-unicorn/issues/990#issuecomment-753474656

Can you open a separate issue for the class member order rule idea? Similar rule for TS: https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/member-ordering.md

Originally posted by @sindresorhus in https://github.com/sindresorhus/eslint-plugin-unicorn/issues/990#issuecomment-755267134

Rule name class-body-order is the original idea, I'm open to better suggestions.

Fail

Cases not in order as example in Pass.

Pass

class Unicorn {
   #privateField = 1;

   publicField= 1;

   static staticField = 1;

   constructor() {}

   #privateMethod() {}

   publicMethod() {}

   static staticMethod() {}

   static {
     // static block
   }
}

Guida contributor

Direzione di ricerca
Studia le regole esistenti in eslint plugin unicorn per la struttura e i pattern di test. Fai riferimento alla regola TypeScript ESLitn all'URL https://github.com/typescript eslint/typescript eslint/blob/master/packages/eslint plugin/docs/rules/member ordering.md per ispirazione. Esamina l'AST dei corpi delle classi utilizzando strumenti come AST Explorer. Implementa la regola con test per i casi di passaggio/fallimento come descritto.
Tech stack
javascripttypescript
Dominio
toolingdeveloper experience
Tipo issue
Funzionalità
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
3-5 giorni
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
ESLint rule developmentAST manipulationJavaScript class syntax
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
50