Refactor cli.rb to be smaller and testable
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
Direzione di ricerca
Inizia leggendo cli.rb per mappare i comandi e le responsabilità attuali. Confronta la struttura con la classe Cli di RuboCop, quindi definisci oggetti comando componibili con un’interfaccia command.run e confini testabili con unit test. Il lavoro è completato quando cli.rb è più piccolo, i comandi sono separati e il refactoring rende più semplice aggiungere unit test.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
As @doodzik mentioned in https://github.com/Shopify/packwerk/pull/61/files#r517545831, we should look into refactoring cli.rb as it is growing to be a large class.
We should move all of the different commands we have in to compositional objects, so we can just do something like command.run. Breaking this class up would make it easier to have unit tests.
I would also be curious to see how RuboCop implements their Cli class.
- Lingua principale
- Ruby
- Stelle
- 1.9k
- Fork
- 125
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
Avvia il container di sviluppo del progetto nel browser, con il tuo account GitHub.
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di Shopify/packwerk
-
[Bug Report] Concurrent runs sharing a cold cache fail with JSON::ParserErrorForse già presa @sarankumar1325 l’ha presa 11 giorni fa. Apertabug
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
-
[Bug Report] Getting "There were stale violations found, please run `packwerk update-todo`" even there is no stale violationsForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertabug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 42/100
Tutte le issue di Shopify/packwerk
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
[Bug] Time search attributes lose sub-second precision, and raise NoMethodError without `require 'time'`Forse già presa @RaphaelFakhri l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
temporalio/sdk-ruby#653 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
bridgetownrb/bridgetown#1134 ·
-
Go: a namespaced model's fixture file is loaded through `NewAdmin::Note()`, which doesn't parseAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
rubys/roundhouse#1820 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
swombat/souls-house#795 ·
I maintainer di solito rispondono entro 1 giorno