Skip to content

Conversation

@Alenar
Copy link
Collaborator

@Alenar Alenar commented Oct 13, 2025

Content

This PR make the mithril-client library re-export the mithril_common::test::double::Dummy trait, allowing third party to use it in their tests.

Note

Re-export of fake_data and fake_keys were considered as well, but were discarded in order to maintain ability to changes them easily without having to worry about breaking the mithril-client library.
This problem is less relevant for the Dummy trait as it's really simple and don't risk to change anytime soon.

Pre-submit checklist

  • Branch
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • All check jobs of the CI have succeeded
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested
  • Documentation
    • No new TODOs introduced

@Alenar Alenar self-assigned this Oct 13, 2025
To allow usage without needing an import of common.

Note: reexport of `fake_data` and `fake_keys` were considered as well
but were discarded in order to maintain ability to changes them easily
without having to worry about breaking `mithril-client` library.
This problem is less relevant for the `Dummy` trait as it's really
simple and don't risk to change anytime soon.
* mithril-client-cli from `0.12.31` to `0.12.32`
* mithril-client from `0.12.31` to `0.12.32`
@Alenar Alenar force-pushed the djo/fix/client-lib/missing-reexport branch from 77cc757 to 7ca4c89 Compare October 13, 2025 12:45
Copy link
Collaborator

@turmelclem turmelclem left a comment

Choose a reason for hiding this comment

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

LGTM 🦒

@github-actions
Copy link

Test Results

    4 files  ±0    168 suites  ±0   24m 25s ⏱️ -3s
2 227 tests ±0  2 227 ✅ ±0  0 💤 ±0  0 ❌ ±0 
6 879 runs  ±0  6 879 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 7ca4c89. ± Comparison against base commit 69e39ed.

@Alenar Alenar temporarily deployed to testing-preview October 13, 2025 12:55 — with GitHub Actions Inactive
Copy link
Member

@jpraynaud jpraynaud left a comment

Choose a reason for hiding this comment

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

LGTM

@Alenar Alenar merged commit 046a4f8 into main Oct 13, 2025
41 checks passed
@Alenar Alenar deleted the djo/fix/client-lib/missing-reexport branch October 13, 2025 15:00
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.

4 participants