Skip to content

Run with debug_assertions enable and less optimization test builds#342

Open
George-Ogden wants to merge 1 commit into
NVlabs:mainfrom
George-Ogden:debug-mode-testing
Open

Run with debug_assertions enable and less optimization test builds#342
George-Ogden wants to merge 1 commit into
NVlabs:mainfrom
George-Ogden:debug-mode-testing

Conversation

@George-Ogden

Copy link
Copy Markdown

Summary

Compile tests with debug_assertions and without optimizations set to level 3.
Fixes #341

Changes

  • only remove debug assertions when compiling non-debug builds
  • only optimize when compiling non-debug builds
  • run tests with debug builds
  • use an enum to represent the cargo subcommand

Testing

Please help with this. I'm not sure of the best way to test this.

  • cargo oxide run <example> passes
  • cargo test --workspace passes
  • New example added (if applicable)

Checklist

  • All commits signed off (git commit -s)
  • SPDX headers on new source files (no new source files)

Questions about the review? Ping us in #contributors on Discord.

Compile without debug_assertions or high-optimization level
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cargo oxide compiles tests without debug_assertions

1 participant