Skip to content

Conversation

@marcusbfs
Copy link
Collaborator

@marcusbfs marcusbfs commented Feb 26, 2025

Closes #1662.

Related PRs:

Pre-review checklist

  • All code has been formatted using our config (make format)
  • Any new API features or modification of existing behavior are covered with tests
  • The template (templates/ctl-scaffold) has been updated
  • The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Changed, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

@marcusbfs marcusbfs force-pushed the marcusbfs/issue-1662/kupmios-extraction branch 3 times, most recently from 9203038 to aacecdf Compare March 3, 2025 10:47
@marcusbfs marcusbfs marked this pull request as ready for review March 7, 2025 18:12
@marcusbfs marcusbfs requested a review from errfrom March 7, 2025 18:12
Copy link
Collaborator

@errfrom errfrom left a comment

Choose a reason for hiding this comment

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

LGTM, lovely ratio of additions to deletions

CHANGELOG.md Outdated
- Switched from WebSocket to HTTP when interfacing with Ogmios. Note: Mempool functionality still uses WebSocket, as it requires a persistent connection to track state. ([#1575](https://github.com/Plutonomicon/cardano-transaction-lib/issues/1575))
- Blockfrost Provider has been extracted to a separate package [purescript-cardano-blockfrost-provider](https://github.com/mlabs-haskell/purescript-cardano-blockfrost-provider) using module names in the format `Cardano.Blockfrost.*` ([#1661](https://github.com/Plutonomicon/cardano-transaction-lib/issues/1661))
- Kupo + Ogmios Provider has been extracted to a separate package [purescript-cardano-kupmios-provider](https://github.com/mlabs-haskell/purescript-cardano-kupmios-provider) using module names in the format `Cardano.Kupmios.*` ([#1662](https://github.com/Plutonomicon/cardano-transaction-lib/issues/1662))
- Ogmios' mempool functionality has been extracted to a separate package [purescript-cardano-ogmios-mempool](https://github.com/mlabs-haskell/purescript-cardano-ogmios-mempool) using the module `Cardano.Ogmios.Mempool`. To use it, the user must create and manage the websocket connection (see [Test.Ctl.Testnet.Contract.OgmiosMempool](https://github.com/Plutonomicon/cardano-transaction-lib/blob/develop/test/Testnet/Contract/OgmiosMempool.purs))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please use permalinks for source code

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Addressed in 7941ac3.

let upstream =
-- https://github.com/mlabs-haskell/purescript-cardano-package-set
https://raw.githubusercontent.com/mlabs-haskell/purescript-cardano-package-set/v1.2.0/packages.dhall sha256:1879aeee12ef41d5f39ed8b530efa817c747366553b2fc90981ad4e8c21fc5d8
https://raw.githubusercontent.com/mlabs-haskell/purescript-cardano-package-set/v1.3.0/packages.dhall
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please release cardano-kupmios-provider v1.0.0 and cardano-ogmios-mempool v1.0.0. Our package set should contain versioned dependencies

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Addressed in defd32f.

@marcusbfs marcusbfs force-pushed the marcusbfs/issue-1662/kupmios-extraction branch from 6a633bc to defd32f Compare March 13, 2025 18:11
@marcusbfs marcusbfs requested a review from errfrom March 13, 2025 18:13
@errfrom errfrom merged commit 7225ec2 into develop Mar 14, 2025
3 checks passed
@errfrom errfrom deleted the marcusbfs/issue-1662/kupmios-extraction branch March 14, 2025 14:24
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.

Move Ogmios+Kupo Provider to a separate package

3 participants