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.

Sign up for GitHub

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

Jump to bottom

Updates 20240121 #573

Merged
merged 7 commits into from
Feb 28, 2024
Merged

Updates 20240121 #573

merged 7 commits into from
Feb 28, 2024

Conversation

louis-langholtz
Copy link
Owner

Description - What's this PR do?

  • Improves Doxygen documentation.
  • Simplifies definitions of Invalid*ID constants.
  • Refactors IsValid functions to use overloading when possible instead of template specialization.
  • Replaces DynamicTree::GetInvalid* functions with Invalid* constant alternatives.

@louis-langholtz louis-langholtz added Enhancement For suggestions or changes that enhance any part of the project and isn't a bug. Docs Use when issue or pull request relates to the documentation of the project. CMake Use to associate with the CMake tool. Library For issues that effect the library and aren't specific to any particular application. labels Feb 26, 2024
@louis-langholtz louis-langholtz added this to the 2.0 Release milestone Feb 26, 2024
@louis-langholtz louis-langholtz self-assigned this Feb 26, 2024
@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@coveralls
Copy link

coveralls commented Feb 26, 2024 •
edited
Loading

Coverage Status

coverage: 99.286%. remained the same
when pulling dde43e7 on updates-20240121
into 5d1c17b on master.

@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@louis-langholtz louis-langholtz force-pushed the updates-20240121 branch 2 times, most recently from b1da710 to 80eacc5 Compare February 27, 2024 05:34
@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

1 similar comment
@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@github-actions GitHub Actions
Copy link
Contributor

clang-tidy review says "All clean, LGTM! 👍"

@louis-langholtz louis-langholtz merged commit 1c5f218 into master Feb 28, 2024
19 checks passed
@louis-langholtz louis-langholtz deleted the updates-20240121 branch February 28, 2024 04:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@louis-langholtz louis-langholtz

Labels
CMake Use to associate with the CMake tool. Docs Use when issue or pull request relates to the documentation of the project. Enhancement For suggestions or changes that enhance any part of the project and isn't a bug. Library For issues that effect the library and aren't specific to any particular application.
Projects
Documentation work
Status: Done
Reduce Namespace Pollution
Status: Done
Milestone
2.0 Release
Development

Successfully merging this pull request may close these issues.
2 participants
@louis-langholtz @coveralls