applied patch from `STR-440-opentelemetry-output`

delbonis commented 21 hours ago

Description

This PR is just applying the patch from #342 onto the release branch. I regenerated the lockfile, and now it's giving me the BlockTransactions issue again. How do we fix this?

Type of Change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature/Enhancement (non-breaking change which adds functionality or enhances an existing one)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Refactor
[ ] New or updated tests

Checklist

[x] I have performed a self-review of my code.
[x] I have commented my code where necessary.
[x] I have updated the documentation if needed.
[ ] My changes do not introduce new warnings.
[ ] I have added tests that prove my changes are effective or that my feature works.
[ ] New and existing tests pass with my changes.

Related Issues

342

storopoli commented 20 hours ago

This is blocked by @evgenyzdanovich's work in removing the reth fork along with @MdTeach. My suggestion is to do reset the Cargo.lock and do cargo update -p FOO --precise BAR. See how BDK does it when dealing with their MSRV breaks: https://github.com/bitcoindevkit/bdk?tab=readme-ov-file#minimum-supported-rust-version-msrv

delbonis commented 20 hours ago

ahhhhhhhhhhhhhhhhhhhhhhhhh

storopoli commented 20 hours ago

closed by #473

alpenlabs / strata