-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
[stdlib] Fix list test warnings #2723
Conversation
Signed-off-by: Brian Grenier <grenierb96@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice fix, thanks!
✅🟣 This contribution has been merged 🟣✅ Your pull request has been merged to the internal upstream Mojo sources. It will be reflected here in the Mojo repository on the nightly branch during the next Mojo nightly release, typically within the next 24-48 hours. We use Copybara to merge external contributions, click here to learn more. |
[External] [stdlib] Fix list test warnings If you break any of the `List` tests you get flooded with warnings and it makes it annoying to find the message telling you what failed. Co-authored-by: bgreni <42788181+bgreni@users.noreply.github.com> Closes #2723 MODULAR_ORIG_COMMIT_REV_ID: 341542be0e0903fbedffdbd8fda3de853b394d7c
Landed in cfc66aa! Thank you for your contribution 🎉 |
[External] [stdlib] Fix list test warnings If you break any of the `List` tests you get flooded with warnings and it makes it annoying to find the message telling you what failed. Co-authored-by: bgreni <42788181+bgreni@users.noreply.github.com> Closes modularml#2723 MODULAR_ORIG_COMMIT_REV_ID: 341542be0e0903fbedffdbd8fda3de853b394d7c
[External] [stdlib] Fix list test warnings If you break any of the `List` tests you get flooded with warnings and it makes it annoying to find the message telling you what failed. Co-authored-by: bgreni <42788181+bgreni@users.noreply.github.com> Closes modularml#2723 MODULAR_ORIG_COMMIT_REV_ID: 341542be0e0903fbedffdbd8fda3de853b394d7c
If you break any of the
List
tests you get flooded with warnings and it makes it annoying to find the message telling you what failed.