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.

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

Add a separate CI job using cargo-careful to test using debug assertion in std. #355

Merged
merged 3 commits into from
Oct 17, 2022

Conversation

adamreichold
Copy link
Member

No description provided.

.github/workflows/ci.yml Outdated Show resolved Hide resolved
@adamreichold adamreichold force-pushed the cargo-careful branch 3 times, most recently from 91c1c66 to a117984 Compare October 16, 2022 14:38
@adamreichold adamreichold changed the title Improve the Valgrind CI job by also using cargo-careful to enable debug assertions in the standard library among other things. Add a separate CI job using cargo-careful to test using debug assertion in std. Oct 16, 2022
@adamreichold
Copy link
Member Author

Combining Valgrind and cargo-careful does not seem to work out. At least the Valgrind errors in the libtest's argument parsing do not seem helpful and do not happen on main.

@adamreichold adamreichold merged commit 2e94120 into main Oct 17, 2022
@adamreichold adamreichold deleted the cargo-careful branch October 17, 2022 06:42
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.

2 participants