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

[v2.0.0] big big big update #5

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

alexv-ds
Copy link

@alexv-ds alexv-ds commented May 23, 2024

Desctiption

Initially I wanted to use it for my project, but decided to give up this library. Therefore I am making PR for future users. PR will be active until I delete my fork, I hope you merge this PR before I do. I am willing to discuss other edits.

You will need to create a variable (CODECOV_TOKEN) before merge
And change references in README.md

It's also a good idea to create a Release after the merge.

What's Changed

  • Fully rewritten cmake project
  • Rewritten tests using the Catch2 library
  • Continuous integration into github-actions has been set up
  • Added analysis of code coverage by tests using gcovr
  • Added valgrind memcheck
  • Fixed corrupted iterator assertions by MSVC
  • Fixed MSVC warnings
  • Fixed memory leak in BlockAllocator
  • Refactoring CamelCase into sneak_case

Full Changelog: alexv-ds/loose_quadtree@v1.0.2...v2.0.0

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.

1 participant