Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Update helpers.rs #6255

Merged
merged 5 commits into from
Sep 11, 2024
Merged

Update helpers.rs #6255

merged 5 commits into from
Sep 11, 2024

Conversation

cypherpepe
Copy link
Contributor

@cypherpepe cypherpepe commented Jul 10, 2024

Simplified conditions and variables for better understanding and readability.

Description

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.

Simplified conditions and variables for better understanding and readability.
@cypherpepe cypherpepe requested a review from a team as a code owner July 10, 2024 14:33
@sdankel
Copy link
Member

sdankel commented Sep 9, 2024

@cypherpepe This looks good to me, but you'll need to run cargo fmt to have all CI checks passing.

@IGI-111 IGI-111 requested a review from a team September 10, 2024 23:41
@sdankel sdankel enabled auto-merge (squash) September 11, 2024 17:14
@sdankel sdankel merged commit 11d8f54 into FuelLabs:master Sep 11, 2024
35 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants