Skip to content

Substitute cryptonite dependency with crypton#5044

Merged
paolino merged 2 commits intomasterfrom
paolino/4996/inline-cardano-sl-509
Mar 13, 2025
Merged

Substitute cryptonite dependency with crypton#5044
paolino merged 2 commits intomasterfrom
paolino/4996/inline-cardano-sl-509

Conversation

@paolino
Copy link
Collaborator

@paolino paolino commented Mar 12, 2025

Changes

  • Import cardano-sl-509 code
  • Ditch cryptonite in favor of crypton

Issues

fix #4996

@paolino paolino force-pushed the paolino/4996/enforce-cabal-fmt branch from 10026d8 to b9a3ff2 Compare March 12, 2025 16:36
@paolino paolino force-pushed the paolino/4996/inline-cardano-sl-509 branch 2 times, most recently from 4cd1283 to 183c1c8 Compare March 12, 2025 17:05
@paolino paolino self-assigned this Mar 12, 2025
@paolino paolino added the Improvement Mark a PR as an improvement, for auto-generated CHANGELOG label Mar 12, 2025
@paolino paolino marked this pull request as ready for review March 12, 2025 17:07
@paolino paolino changed the title Import cardano-sl-509 code Substitute cryptonite dependency with crypton Mar 12, 2025
@paolino paolino force-pushed the paolino/4996/inline-cardano-sl-509 branch from 183c1c8 to a630274 Compare March 12, 2025 17:43
Base automatically changed from paolino/4996/enforce-cabal-fmt to master March 12, 2025 20:06
Copy link
Collaborator

@paweljakubas paweljakubas left a comment

Choose a reason for hiding this comment

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

looks valid provided x509 was relocated intact :-)

@paolino
Copy link
Collaborator Author

paolino commented Mar 13, 2025

looks valid provided x509 was relocated intact :-)

I removed cardano-prelude as a dependency and apply hints. I will double check.

@paolino paolino merged commit 0652580 into master Mar 13, 2025
32 checks passed
@paolino paolino deleted the paolino/4996/inline-cardano-sl-509 branch March 13, 2025 11:03
@paolino paolino restored the paolino/4996/inline-cardano-sl-509 branch March 13, 2025 11:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Improvement Mark a PR as an improvement, for auto-generated CHANGELOG

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Migrate dependency on x509 to crypton-x509

2 participants