Skip to content

Conversation

nbacquey
Copy link
Contributor

@nbacquey nbacquey commented Aug 21, 2025

Since createTestnetEnv is now responsible for creating the sandbox environment, cardanoTestnet no longer needs Genesis information. This commit removes the dependency, and gets the needed information from the sandbox environment instead.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • CI passes. See note on CI. The following CI checks are required:
    • Code is linted with hlint. See .github/workflows/check-hlint.yml to get the hlint version
    • Code is formatted with stylish-haskell. See .github/workflows/stylish-haskell.yml to get the stylish-haskell version
    • Code builds on Linux, MacOS and Windows for ghc-9.6 and ghc-9.12
  • Self-reviewed the diff

@nbacquey nbacquey requested a review from a team as a code owner August 21, 2025 16:09
@nbacquey nbacquey force-pushed the nb/refactor_genesis branch 2 times, most recently from f6c0b83 to a2c71dc Compare August 25, 2025 13:54
@nbacquey nbacquey requested review from a team as code owners August 25, 2025 13:54
@nbacquey nbacquey changed the base branch from master to nb/remove_broken_link August 25, 2025 13:54
@nbacquey nbacquey force-pushed the nb/remove_broken_link branch from 20c8407 to 46cbb65 Compare August 27, 2025 12:08
@nbacquey nbacquey force-pushed the nb/refactor_genesis branch from a2c71dc to 924729e Compare August 27, 2025 12:10
Copy link
Contributor

@carbolymer carbolymer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Since `createTestnetEnv` is now responsible for creating the sandbox
environment, `cardanoTestnet` no longer needs Genesis information. This
commit removes the dependency, and gets the needed information from the
sandbox environment instead.
@nbacquey nbacquey force-pushed the nb/refactor_genesis branch from 924729e to 205c733 Compare August 27, 2025 13:50
@nbacquey nbacquey merged commit e289c67 into nb/remove_broken_link Aug 27, 2025
12 of 14 checks passed
@nbacquey nbacquey deleted the nb/refactor_genesis branch August 27, 2025 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants