Clean up mkConf / mkConfig / mkConfigAbs naming in cardano-testnet
I maintainer di solito rispondono entro 1 giorno
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Testnet.Start.Types has three functions for creating Conf values:
mkConf :: (HasCallStack, MonadTest m) => FilePath -> m Conf- annotates the path in test output, then callsmkConfigmkConfig :: FilePath -> Conf- pure, adds trailing separator and defaultsmkConfigAbs :: FilePath -> IO Conf- makes path absolute, creates directory, callsmkConfig
The names do not communicate the differences.
mkConf vs mkConfig looks like a typo rather than a meaningful distinction.
mkConfigAbs conflates "make path absolute" with "run in IO".
Suggested cleanup (open to better ideas):
mkConfigstays as the pure coremkConfbecomes something likemkConfigForTestormkConfigAnnotatedto signal theMonadTestdependencymkConfigAbsbecomes something likemkConfigAbsoluteormkConfigIO
Or collapse them if the separate variants are not pulling their weight.
Context: Flagged by @carbolymer in https://github.com/IntersectMBO/cardano-node/pull/6346 review comments.
- Lingua principale
- Haskell
- Stelle
- 3.2k
- Fork
- 762
- Merge medio
- 1g 23h
- PR unite (30g)
- 21
Preparare l'ambiente
- Include un Dockerfile o un 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 IntersectMBO/cardano-node
-
Stale
Difficoltà 1/5 1-3 ore Idoneità per principianti 75/100
IntersectMBO/cardano-node#6597 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Stale
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
IntersectMBO/cardano-node#6578 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
IntersectMBO/cardano-node#6522 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
needs triage
Difficoltà 4/5 3-5 giorni Idoneità per principianti 44/100
IntersectMBO/cardano-node#6734 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
increase in memory usageAperta
Difficoltà 4/5 3-5 giorni Idoneità per principianti 28/100
IntersectMBO/cardano-node#6718 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di IntersectMBO/cardano-node
Issue simili
-
Update duckdb-simple to 0.3Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
haskell-beam/beam#828 ·
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
simplex-chat/simplex-chat#7755 ·
I maintainer di solito rispondono entro 1 giorno
-
bug core-libs
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
-
content good first issue
Difficoltà 2/5 Meno di un'ora Idoneità per principianti 88/100
StudentSuite/awesome-skills-plugins-for-students#312 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 96/100
alunduil/collection-json.hs#415 ·
I maintainer di solito rispondono entro 1 giorno