Hacktoberfest 2026: những issue maintainer đã đánh dấu cho tháng Mười, đang mở và phù hợp người mới. Xem issue Hacktoberfest

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

Đang mở
  • Có gì không ổn?
#6,569 2 bình luận 0 reaction 1 người được giao Xem trên GitHub

Maintainer thường phản hồi trong vòng 1 ngày

@palas đang làm issue này rồi.

Từ ngày 4/8/2026.

  • #6637 của @palas — đang mở

Đánh giá

Issue này chưa được đánh giá.

Mô tả

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.

Ngôn ngữ chính
Haskell
Star
3.2k
Fork
760
Merge trung bình
2 ngày 7 giờ
Pull request đã merge (30 ngày)
21

Chuẩn bị môi trường

Bắt đầu từ đâu

  1. Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
  2. Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
  3. Fork repository và làm thay đổi trên một nhánh.
  4. Mở pull request có tham chiếu số hiệu của issue.

Issue khác của IntersectMBO/cardano-node

Tất cả issue của IntersectMBO/cardano-node

Issue tương tự

Thêm issue về Haskell

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.