Skip to content

fix: update tap, axum, tonic, alloy dependencies: #702

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 6 commits into from
Apr 28, 2025
Merged

fix: update tap, axum, tonic, alloy dependencies: #702

merged 6 commits into from
Apr 28, 2025

Conversation

suchapalaver
Copy link
Collaborator

@suchapalaver suchapalaver commented Apr 23, 2025 •
edited
Loading

  • fix wiremock-grpc
  • update axum
  • update tonic
  • update tap_ crates
  • update thegraph_core
    ...
  • fix issues in code now broken by axum update
  • fix tests failing by running tests talking to a database instance serially

Signed-off-by: Joseph Livesey [email protected]

@suchapalaver suchapalaver changed the title feat: update tap, axum, tonic, alloy dependencies: wip: update tap, axum, tonic, alloy dependencies: Apr 23, 2025
@suchapalaver suchapalaver force-pushed the suchapalaver/update-dependencies branch 3 times, most recently from d9c6c95 to 81d12c9 Compare April 23, 2025 01:17
@suchapalaver suchapalaver mentioned this pull request Apr 23, 2025
Closed
@suchapalaver suchapalaver force-pushed the suchapalaver/update-dependencies branch 3 times, most recently from c5c7602 to 73c4e2d Compare April 28, 2025 20:50
@github-actions GitHub Actions
Copy link
Contributor

Pull Request Test Coverage Report for Build 14717598963

Details

  • 14 of 27 (51.85%) changed or added relevant lines in 7 files are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.08%) to 74.037%
Changes Missing Coverage Covered Lines Changed/Added Lines %
crates/service/src/service/router.rs 4 5 80.0%
crates/dips/src/proto/graphprotocol.gateway.dips.rs 0 6 0.0%
crates/dips/src/proto/graphprotocol.indexer.dips.rs 0 6 0.0%
Files with Coverage Reduction New Missed Lines %
crates/service/src/tap/receipt_store.rs 2 94.89%
Totals Coverage Status
Change from base Build 14696959450: -0.08%
Covered Lines: 8766
Relevant Lines: 11840
💛 - Coveralls

@suchapalaver suchapalaver marked this pull request as ready for review April 28, 2025 21:08
@suchapalaver suchapalaver force-pushed the suchapalaver/update-dependencies branch from 73c4e2d to b5cd478 Compare April 28, 2025 21:16
@suchapalaver suchapalaver enabled auto-merge (squash) April 28, 2025 21:17
@suchapalaver suchapalaver changed the title wip: update tap, axum, tonic, alloy dependencies: chore: update tap, axum, tonic, alloy dependencies: Apr 28, 2025
@suchapalaver suchapalaver changed the title chore: update tap, axum, tonic, alloy dependencies: fix: update tap, axum, tonic, alloy dependencies: Apr 28, 2025
@suchapalaver suchapalaver disabled auto-merge April 28, 2025 21:18
@suchapalaver suchapalaver enabled auto-merge (squash) April 28, 2025 21:18
neithanmo
neithanmo approved these changes Apr 28, 2025
View reviewed changes
Copy link
Collaborator

@neithanmo neithanmo left a comment

Choose a reason for hiding this comment

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

Great!

@suchapalaver suchapalaver merged commit 6e49c0c into main Apr 28, 2025
10 checks passed
@suchapalaver suchapalaver deleted the suchapalaver/update-dependencies branch April 28, 2025 21:37
@github-actions github-actions bot mentioned this pull request Apr 28, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@neithanmo neithanmo neithanmo approved these changes

@TypeLevelConsoli TypeLevelConsoli Awaiting requested review from TypeLevelConsoli

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@suchapalaver @neithanmo