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

Refactor tests #32

Merged
merged 9 commits into from
Jan 5, 2025
Merged

Refactor tests #32

merged 9 commits into from
Jan 5, 2025

Conversation

pedrobiqua
Copy link
Collaborator

Pull Request

Checklist

Please ensure that your PR fills the following requirements:

  • Unit tests have been added or updated for the changes (for bugfixes or new features).
  • Code compiles without warnings or errors.
  • All new and existing tests pass locally.
  • Code follows the project's C++ style guidelines (e.g., naming conventions, indentation).
  • Documentation has been updated where applicable (e.g., function headers, inline comments).

PR Type

What type of change does this PR introduce?

  • Bugfix
  • New feature
  • Performance improvement
  • Code style update (formatting, naming conventions)
  • Refactoring (no functional or API changes)
  • Build or configuration changes
  • CI-related changes
  • Documentation updates
  • Other (please describe):

What is the current behavior?

Issue Number: #31

Does this PR introduce a breaking change?

  • Yes
  • No

@pedrobiqua pedrobiqua linked an issue Jan 5, 2025 that may be closed by this pull request
5 tasks
@pedrobiqua pedrobiqua closed this Jan 5, 2025
@pedrobiqua pedrobiqua reopened this Jan 5, 2025
@pedrobiqua pedrobiqua merged commit 5bf2f45 into main Jan 5, 2025
6 checks passed
@pedrobiqua pedrobiqua deleted the refactor-tests branch January 5, 2025 18:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Refactor] Separate Build and Test in CMake
1 participant