Skip to content

Testing the test framework - Catch2#14

Closed
robert-andrzejuk wants to merge 93 commits intotest-framework-checkfrom
testing-test
Closed

Testing the test framework - Catch2#14
robert-andrzejuk wants to merge 93 commits intotest-framework-checkfrom
testing-test

Conversation

@robert-andrzejuk
Copy link
Collaborator

No description provided.

@JeffGarland
Copy link
Member

ok well boost.ut does the same as catch2 without macros, but I'm good -- anything but gtest. Approving this PR

JeffGarland and others added 22 commits April 19, 2025 06:30
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
);

res.reset(); // disables cleanup, sets cleaned true

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[pre-commit] reported by reviewdog 🐶

Suggested change

res.reset(); // disables cleanup, sets cleaned true

cleaned = false; //reset cleaned

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[pre-commit] reported by reviewdog 🐶

Suggested change

@JeffGarland
Copy link
Member

I'm closing this PR since it's running CI for no real reason and I'm working on the branch in #17

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.

5 participants

Comments