Hacktoberfest 2026：メンテナが10月に向けて印を付けた、オープンで初心者向けの issue。 Hacktoberfest の issue を見る

Clean up mkConf / mkConfig / mkConfigAbs naming in cardano-testnet

オープン
  • 何か問題がありますか？
#6,569 コメント 2 件 リアクション 0 件 担当者 1 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

@palas がすでに取り組んでいます。

2026年8月4日 から。

  • #6637 @palas による — オープン

評価

この issue はまだ評価されていません。

説明

Testnet.Start.Types has three functions for creating Conf values:

  • mkConf :: (HasCallStack, MonadTest m) => FilePath -> m Conf - annotates the path in test output, then calls mkConfig
  • mkConfig :: FilePath -> Conf - pure, adds trailing separator and defaults
  • mkConfigAbs :: FilePath -> IO Conf - makes path absolute, creates directory, calls mkConfig

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):

  • mkConfig stays as the pure core
  • mkConf becomes something like mkConfigForTest or mkConfigAnnotated to signal the MonadTest dependency
  • mkConfigAbs becomes something like mkConfigAbsolute or mkConfigIO

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.

主要言語
Haskell
スター
3.2k
フォーク
760
平均マージ
2日 7時間
マージ済み PR（30日）
21

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

IntersectMBO/cardano-node のほかの issue

IntersectMBO/cardano-node の issue をすべて見る

似ている issue

Haskell の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。