-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Implement the Eq
trait for Option
#3542
Labels
Comments
I don't believe this is blocked anymore. Will see what I can do. |
7 tasks
bitzoic
added a commit
that referenced
this issue
Dec 3, 2023
## Description Closes #3542 Implements the `Eq` trait for `Option`. Also adds tests for the implementation. ## Checklist - [x] I have linked to any relevant issues. - [x] 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). - [x] 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. - [x] I have done my best to ensure that my PR adheres to [the Fuel Labs Code Review Standards](https://github.com/FuelLabs/rfcs/blob/master/text/code-standards/external-contributors.md). - [x] I have requested a review from the relevant team or maintainers. --------- Co-authored-by: Braqzen <103777923+Braqzen@users.noreply.github.com> Co-authored-by: Cameron Carstens <54727135+bitzoic@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The
Identity
implements theEq
trait:sway/sway-lib-std/src/identity.sw
Lines 13 to 21 in 716b34f
I would like the same thing to exist for
Option
so I can compare Options.The text was updated successfully, but these errors were encountered: