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 information #1674

Merged
merged 5 commits into from
May 21, 2024

Conversation

falcucci
Copy link
Collaborator

@falcucci falcucci commented May 11, 2024

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)

    fixes ChainObserver supports retrieving the Chain Point of the tip of the chain #1589

- 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 changed the title feat: add a new get_current_chain_point function to fetch chain point information feat: add get_current_chain_point function to fetch chain point information May 11, 2024
@falcucci falcucci changed the title feat: add get_current_chain_point function to fetch chain point information WIP feat: add get_current_chain_point function to fetch chain point information May 11, 2024
@falcucci falcucci changed the title WIP feat: add get_current_chain_point function to fetch chain point information feat: add get_current_chain_point function to fetch chain point information May 11, 2024
@falcucci falcucci marked this pull request as draft May 11, 2024 00:14
- remove unnecessary commented out paths for dependencies
- update versions of dependencies
- reorganize feature flags for the `criterion` dependency
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 👍

Thanks for the PR @falcucci 🙂

@falcucci
Copy link
Collaborator Author

@jpraynaud thanks! I'll leave this as draft until we have pallas released, ping you up when I bump the version.

@ch1bo
Copy link
Member

ch1bo commented May 14, 2024

@jpraynaud thanks! I'll leave this as draft until we have pallas released, ping you up when I bump the version.

Which PR would that be upstream?

@falcucci
Copy link
Collaborator Author

falcucci commented May 14, 2024

@ch1bo from pallas side, that one and about this one, I'll actually remove it from draft right now 🙂

@falcucci falcucci marked this pull request as ready for review May 14, 2024 13:07
- 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`
@jpraynaud jpraynaud merged commit d0f4c91 into input-output-hk:main May 21, 2024
39 checks passed
@falcucci falcucci self-assigned this May 23, 2024
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.

ChainObserver supports retrieving the Chain Point of the tip of the chain
4 participants