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

chore: upgrade prost and tonic (ring dep) #6603

Merged
merged 1 commit into from
Oct 4, 2024

Conversation

sdbondi
Copy link
Member

@sdbondi sdbondi commented Oct 4, 2024

Description

chore: upgrade prost and tonic (ring dep)

Motivation and Context

Part of the way to removing ring 0.16. rcgen still required.

How Has This Been Tested?

What process can a PR reviewer use to test or verify this change?

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

@sdbondi sdbondi requested a review from a team as a code owner October 4, 2024 07:44
@ghpbot-tari-project ghpbot-tari-project added P-acks_required Process - Requires more ACKs or utACKs P-reviews_required Process - Requires a review from a lead maintainer to be merged labels Oct 4, 2024
Copy link

github-actions bot commented Oct 4, 2024

Test Results (CI)

    3 files    129 suites   34m 47s ⏱️
1 325 tests 1 325 ✅ 0 💤 0 ❌
3 973 runs  3 973 ✅ 0 💤 0 ❌

Results for commit 827fc40.

Copy link

github-actions bot commented Oct 4, 2024

Test Results (Integration tests)

 2 files  + 2  11 suites  +11   17m 15s ⏱️ + 17m 15s
36 tests +36  35 ✅ +35  0 💤 ±0  1 ❌ +1 
37 runs  +37  36 ✅ +36  0 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 827fc40. ± Comparison against base commit 5bcddf4.

@SWvheerden SWvheerden merged commit f7f7bb5 into tari-project:development Oct 4, 2024
16 of 18 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-acks_required Process - Requires more ACKs or utACKs P-reviews_required Process - Requires a review from a lead maintainer to be merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants