Skip to content

Conversation

@Alenar
Copy link
Collaborator

@Alenar Alenar commented Nov 17, 2025

Content

This PR remove the crate-type field from all crates except mithril-stm and mithril-client (and mithril-client-wasm where the crate-type = ["cdylib"] is mandatory for compilation).

Addition of staticlib add a lot of disk usage and build time and is not reasonable for crates that are not intended to be directly used by third parties.

Pre-submit checklist

  • Branch
    • Crates versions are updated (if relevant)
    • CHANGELOG file is 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

Issue(s)

Relates to #2782

Addition of `staticlib` add a lot of disk usage and build time and is
not justified for crates that are not intended to be directly used by
third parties (all crates except `mithril-stm` and `mithril-client`).
* mithril-cardano-node-chain from `0.1.9` to `0.1.10`
* mithril-cardano-node-internal-database from `0.1.7` to `0.1.8`
* mithril-aggregator-client from `0.1.2` to `0.1.3`
* mithril-dmq from `0.1.13` to `0.1.14`
* mithril-era from `0.1.5` to `0.1.6`
* mithril-metric from `0.1.19` to `0.1.20`
* mithril-persistence from `0.2.59` to `0.2.60`
* mithril-resource-pool from `0.0.8` to `0.0.9`
* mithril-ticker from `0.1.3` to `0.1.4`
* mithril-signed-entity-lock from `0.0.4` to `0.0.5`
* mithril-signed-entity-preloader from `0.0.10` to `0.0.11`
* mithril-common from `0.6.27` to `0.6.28`
@Alenar Alenar requested a review from jpraynaud November 17, 2025 10:29
@Alenar Alenar self-assigned this Nov 17, 2025
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 👍

@github-actions
Copy link

github-actions bot commented Nov 17, 2025

Test Results

    4 files  ±0    168 suites  ±0   22m 15s ⏱️ - 1m 51s
2 210 tests ±0  2 210 ✅ ±0  0 💤 ±0  0 ❌ ±0 
6 895 runs  ±0  6 895 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 1c11101. ± Comparison against base commit 59bf453.

♻️ This comment has been updated with latest results.

@Alenar Alenar temporarily deployed to testing-preview November 17, 2025 11:05 — with GitHub Actions Inactive
@Alenar Alenar merged commit b9f2466 into main Nov 17, 2025
70 of 71 checks passed
@Alenar Alenar deleted the djo/remove_uneeded_crates_type branch November 17, 2025 11:06
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