Skip to content

Conversation

@jpraynaud
Copy link
Member

@jpraynaud jpraynaud commented Nov 4, 2024

Content

This PR upgrades the repository dependencies to their latest version in preparation of the distribution 2445 by running the upgrade script from the runbook.

Dependency updates:

  • Updated serde to version 1.0.214 and serde_json to version 1.0.132 in demo/protocol-demo/Cargo.toml, mithril-persistence/Cargo.toml, mithril-aggregator/Cargo.toml, mithril-client-cli/Cargo.toml, and mithril-client-wasm/Cargo.toml. [1] [2] [3] [4] [5]

  • Updated tokio to version 1.41.0 in examples/client-cardano-stake-distribution/Cargo.toml, examples/client-cardano-transaction/Cargo.toml, examples/client-mithril-stake-distribution/Cargo.toml, examples/client-snapshot/Cargo.toml, mithril-aggregator/Cargo.toml, mithril-persistence/Cargo.toml, and mithril-client-cli/Cargo.toml. [1] [2] [3] [4] [5] [6] [7]

  • Updated anyhow to version 1.0.92 in mithril-aggregator/Cargo.toml, mithril-client-cli/Cargo.toml, mithril-client-wasm/Cargo.toml, mithril-metric/Cargo.toml, and mithril-persistence/Cargo.toml. [1] [2] [3] [4] [5]

Version updates:

Documentation and scripts:

  • Fixed a typo in docs/devbook/upgrade-repository-dependencies/upgrade_dependencies.sh and reordered commands for clarity.

  • Updated dependencies in docs/website/package.json to the latest versions.

Pre-submit checklist

  • Branch
    • Crates versions are updated (if relevant)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
  • PR
    • No clippy warnings in the CI
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

Issue(s)

Relates to #2030

@jpraynaud jpraynaud self-assigned this Nov 4, 2024
@github-actions
Copy link

github-actions bot commented Nov 4, 2024

Test Results

    4 files  ±0     51 suites  ±0   10m 39s ⏱️ +3s
1 418 tests ±0  1 418 ✅ ±0  0 💤 ±0  0 ❌ ±0 
1 629 runs  ±0  1 629 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 58219c4. ± Comparison against base commit ef41854.

♻️ This comment has been updated with latest results.

@jpraynaud jpraynaud force-pushed the jpraynaud/2030-update-dependencies-distribution-2445 branch from 5e9b82b to 7900e70 Compare November 4, 2024 14:04
@jpraynaud jpraynaud temporarily deployed to testing-sanchonet November 4, 2024 14:18 — with GitHub Actions Inactive
@jpraynaud jpraynaud force-pushed the jpraynaud/2030-update-dependencies-distribution-2445 branch from 7900e70 to 732efd5 Compare November 4, 2024 15:38
@jpraynaud jpraynaud temporarily deployed to testing-sanchonet November 4, 2024 15:52 — with GitHub Actions Inactive
Copy link
Collaborator

@dlachaume dlachaume left a comment

Choose a reason for hiding this comment

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

LGTM 👍

'pallas-codec' '0.31' is not compatible with other pallas crates '0.30'.
By running command 'cargo update -p [email protected] --precise 1.1.20'.
By running 'make upgrade' command.
By running 'npm audit fix'.
By running 'make upgrade' command.
By running:
- 'make www-install' command in 'mithril-client-wasm'.
- 'make www-test-install' command in 'mithril-client-wasm'.
- 'make install' command in 'mithril-explorer'.
- 'make install' command in 'docs/website'.
 By running 'nix flake update' command.
@jpraynaud jpraynaud force-pushed the jpraynaud/2030-update-dependencies-distribution-2445 branch from 732efd5 to ed04a97 Compare November 5, 2024 09:16
@jpraynaud jpraynaud force-pushed the jpraynaud/2030-update-dependencies-distribution-2445 branch from ed04a97 to 58219c4 Compare November 5, 2024 09:31
@jpraynaud jpraynaud temporarily deployed to testing-sanchonet November 5, 2024 09:45 — with GitHub Actions Inactive
@jpraynaud jpraynaud merged commit fcca57d into main Nov 5, 2024
47 checks passed
@jpraynaud jpraynaud deleted the jpraynaud/2030-update-dependencies-distribution-2445 branch November 5, 2024 10: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.

5 participants