Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Conversation

brooksprumo
Copy link
Contributor

Problem

While upgrading Rust in #32961, we see that the panic message format has changed in Rust 1.73.0+. This causes a bucket storage test to fail erroneously.

Summary of Changes

Refactor the test to remove the text check on the panic message.

Copy link
Contributor

@HaoranYi HaoranYi left a comment

Choose a reason for hiding this comment

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

lgtm

@brooksprumo brooksprumo added the automerge Merge this Pull Request automatically once CI passes label Aug 31, 2023
@brooksprumo brooksprumo marked this pull request as ready for review August 31, 2023 17:00
@brooksprumo brooksprumo removed the automerge Merge this Pull Request automatically once CI passes label Aug 31, 2023
@brooksprumo brooksprumo merged commit f8789c7 into solana-labs:master Aug 31, 2023
@brooksprumo brooksprumo deleted the bucket-map/bucket-header-test branch August 31, 2023 17:01
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants