Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Update docs mentioning tests should be run in release mode #36

Closed
alexfertel opened this issue Apr 5, 2024 · 1 comment
Closed

Update docs mentioning tests should be run in release mode #36

alexfertel opened this issue Apr 5, 2024 · 1 comment
Assignees
@alexfertel
Labels
effort: low Easy or tiny task that takes less than a day. good first issue Good for newcomers. priority: 1 We will do our best to deal with this. type: test Changes to the testing suite.

Comments

@alexfertel
Copy link
Contributor

Given that certain behavior only triggers in release mode in Rust like arithmetic operations wrapping, we should run tests in release mode.

This issue reflects the work needed to update all docs to mention the above.
@alexfertel alexfertel added priority: 1 We will do our best to deal with this. type: test Changes to the testing suite. good first issue Good for newcomers. effort: low Easy or tiny task that takes less than a day. labels Apr 5, 2024
@alexfertel alexfertel self-assigned this Jun 2, 2024
@alexfertel
Copy link
Contributor Author

Closing in favor of #189

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alexfertel alexfertel

Labels
effort: low Easy or tiny task that takes less than a day. good first issue Good for newcomers. priority: 1 We will do our best to deal with this. type: test Changes to the testing suite.
Projects
Rust Contracts Stylus
Status: Done
Milestone
No milestone
Development

No branches or pull requests
1 participant
@alexfertel