Skip to content

Add migration for Bytes::into(self) -> b256 to Bytes::try_into #7009

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

Merged
merged 2 commits into from
Mar 11, 2025

Conversation

ironcev
Copy link
Member

@ironcev ironcev commented Mar 11, 2025

Description

This PR adds migration step for migrating <bytes>.into() calls to <bytes>.try_into().unwrap, as explained in the Breaking Changes chapter of #6994.

Checklist

  • I have linked to any relevant issues.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have updated the documentation where relevant (API docs, the reference, and the Sway book).
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added (or requested a maintainer to add) the necessary Breaking* or New Feature labels where relevant.
  • I have done my best to ensure that my PR adheres to the Fuel Labs Code Review Standards.
  • I have requested a review from the relevant team or maintainers.

@ironcev ironcev self-assigned this Mar 11, 2025
@ironcev ironcev added lib: std Standard library forc-migrate Everything to do with the forc-migrate tool. labels Mar 11, 2025
@ironcev ironcev marked this pull request as ready for review March 11, 2025 08:33
@ironcev ironcev requested a review from a team as a code owner March 11, 2025 08:33
@ironcev ironcev enabled auto-merge (squash) March 11, 2025 08:33
@ironcev ironcev requested review from bitzoic and a team March 11, 2025 08:34
@ironcev ironcev merged commit 1f407fa into master Mar 11, 2025
40 checks passed
@ironcev ironcev deleted the ironcev/migration-for-bytes-try-into-b256 branch March 11, 2025 10:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
forc-migrate Everything to do with the forc-migrate tool. lib: std Standard library
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants