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

Define reflection for glm types #606

Merged
merged 14 commits into from
Sep 28, 2023
Merged

Conversation

luishfonseca
Copy link
Contributor

@luishfonseca luishfonseca commented Sep 27, 2023

Description

Implement Constructible and Fields traits for all glm types

Checklist

  • Self-review changes.
  • Evaluate impact on the documentation.
  • Ensure test coverage.

@luishfonseca luishfonseca added this to the 0.0 Nursery milestone Sep 27, 2023
@luishfonseca luishfonseca self-assigned this Sep 27, 2023
@luishfonseca luishfonseca linked an issue Sep 27, 2023 that may be closed by this pull request
@codecov
Copy link

codecov bot commented Sep 27, 2023

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (c12921f) 36.29% compared to head (fe2d258) 36.57%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #606      +/-   ##
==========================================
+ Coverage   36.29%   36.57%   +0.28%     
==========================================
  Files          88       89       +1     
  Lines        6318     6357      +39     
==========================================
+ Hits         2293     2325      +32     
- Misses       4025     4032       +7     
Files Coverage Δ
core/src/cubos/core/reflection/external/glm.cpp 82.05% <82.05%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@luishfonseca luishfonseca force-pushed the 539-define-reflection-for-glm-types branch 2 times, most recently from 788bc42 to b1a5ea9 Compare September 28, 2023 10:23
@github-actions
Copy link
Contributor

github-actions bot commented Sep 28, 2023

PR Preview Action v1.4.4
Preview removed because the pull request was closed.
2023-09-28 17:39 UTC

Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ clang-tidy found issue(s) with the introduced code (1/2)

core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

⚠️ clang-tidy found issue(s) with the introduced code (2/2)

core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
@github-actions github-actions bot dismissed stale reviews from themself September 28, 2023 12:57

No clang-tidy warnings found so I assume my comments were addressed

@luishfonseca luishfonseca force-pushed the 539-define-reflection-for-glm-types branch from 8465b31 to 6af58ef Compare September 28, 2023 12:57
@luishfonseca luishfonseca marked this pull request as ready for review September 28, 2023 14:04
@luishfonseca luishfonseca force-pushed the 539-define-reflection-for-glm-types branch 3 times, most recently from c33a627 to b01ce1f Compare September 28, 2023 14:25
.githooks/pre-commit Outdated Show resolved Hide resolved
core/include/cubos/core/reflection/external/glm.hpp Outdated Show resolved Hide resolved
core/src/cubos/core/reflection/external/glm.cpp Outdated Show resolved Hide resolved
@luishfonseca luishfonseca force-pushed the 539-define-reflection-for-glm-types branch from 424360c to 6716ade Compare September 28, 2023 14:42
@luishfonseca luishfonseca force-pushed the 539-define-reflection-for-glm-types branch from 6ec40f3 to 3d54b8c Compare September 28, 2023 17:13
@luishfonseca luishfonseca merged commit e8ad837 into main Sep 28, 2023
9 checks passed
@luishfonseca luishfonseca deleted the 539-define-reflection-for-glm-types branch September 28, 2023 17:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Define reflection for glm types
2 participants