Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add get_current_chain_point function to fetch chain point info #5

Merged
merged 5 commits into from
May 22, 2024

Conversation

falcucci
Copy link

Content

This PR includes...

Pre-submit checklist

  • Branch
    • Tests are provided (if possible)
    • Crates versions are updated (if relevant)
    • CHANGELOG file is 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
  • Documentation
    • Update README file (if relevant)
    • Update documentation website (if relevant)
    • Add dev blog post (if relevant)

- update the source url of the pallas-addresses package
- update the source url of the pallas-codec package
- update the source url of the pallas-hardano package
- update the source url of the pallas-network package
- update the source url of the pallas-primitives package
- update the source url of the pallas-traverse package
- update the source url of the pallas-crypto package
- modify the `do_get_chain_block_no` function to async and handle errors
- add a new `get_chain_point` function to fetch chain point information
- implement the `get_current_chain_point` function for pallas
- add a new `get_current_chain_point` test function
@falcucci falcucci self-assigned this May 11, 2024
@falcucci falcucci requested a review from scarmuega May 11, 2024 00:18
Copy link

github-actions bot commented May 11, 2024

Test Results

    3 files  ±0     43 suites  ±0   8m 35s ⏱️ +10s
  971 tests +1    971 ✅ +1  0 💤 ±0  0 ❌ ±0 
1 065 runs  +1  1 065 ✅ +1  0 💤 ±0  0 ❌ ±0 

Results for commit 68735e6. ± Comparison against base commit 9104258.

♻️ This comment has been updated with latest results.

- remove unnecessary commented out paths for dependencies
- update versions of dependencies
- reorganize feature flags for the `criterion` dependency
@falcucci falcucci changed the title feat: add get_current_chain_point function to fetch chain point information feat: add get_current_chain_point function to fetch chain point info May 12, 2024
falcucci added 3 commits May 21, 2024 14:30
- update package versions in `cargo.lock` from `0.25.0` to `0.26.0`
- change package sources in `cargo.lock` to use `crates.io-index`
- update package versions in `cargo.toml` in `mithril-common` directory from `0.25.0` to `0.26.0`
- add new dependencies with updated versions in `cargo.toml` in `mithril-common` directory
- update version of `mithril-common` from 0.4.4 to 0.4.5 in `cargo.lock` and `mithril-common/cargo.toml`
@falcucci falcucci merged commit d0f4c91 into main May 22, 2024
39 checks passed
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.

1 participant