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

Added new macros #37

Merged
merged 17 commits into from
Aug 20, 2023
Merged

Added new macros #37

merged 17 commits into from
Aug 20, 2023

Conversation

marvin-hansen
Copy link
Member

Describe your changes

Added new macros:

  • Constructur
  • Getters

Updated internal types to use macros to generate boilerplate code.

Issue ticket number and link

Code checklist before requesting a review

  • I have signed the DCO?
  • [ x] All tests are passing when running make test?
  • [x ] No errors or security vulnerabilities are reported by make check?

For details on make, please see BUILD.md

Note: The CI runs all of the above and fixing things before they hit CI speeds
up the review and merge process. Thank you.

@marvin-hansen marvin-hansen added the enhancement New feature or request label Aug 20, 2023
@marvin-hansen marvin-hansen self-assigned this Aug 20, 2023
@codecov
Copy link

codecov bot commented Aug 20, 2023

Codecov Report

Patch coverage: 80.09% and project coverage change: -1.66% ⚠️

Comparison is base (3b3dc9b) 91.72% compared to head (dbc8741) 90.06%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
- Coverage   91.72%   90.06%   -1.66%     
==========================================
  Files          87       90       +3     
  Lines        2682     2890     +208     
==========================================
+ Hits         2460     2603     +143     
- Misses        222      287      +65     
Files Changed Coverage Δ
deep_causality_macros/src/constructor.rs 71.02% <71.02%> (ø)
deep_causality_macros/src/getters.rs 86.76% <86.76%> (ø)
...es/context_types/adjustable/adjustable_data/mod.rs 100.00% <100.00%> (+12.50%) ⬆️
...es/context_types/adjustable/adjustable_time/mod.rs 100.00% <100.00%> (+7.69%) ⬆️
...lity/src/types/context_types/node_types/dateoid.rs 100.00% <100.00%> (+7.69%) ⬆️
...usality/src/types/context_types/node_types/root.rs 100.00% <100.00%> (+8.33%) ⬆️
...rc/types/context_types/node_types/space_tempoid.rs 100.00% <100.00%> (+2.85%) ⬆️
...ity/src/types/context_types/node_types/spaceoid.rs 100.00% <100.00%> (+4.16%) ⬆️
...lity/src/types/context_types/node_types/tempoid.rs 100.00% <100.00%> (+4.34%) ⬆️
deep_causality/src/types/csm_types/csm_action.rs 100.00% <100.00%> (+7.14%) ⬆️
... and 7 more

... and 1 file with indirect coverage changes

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

Signed-off-by: Marvin Hansen <[email protected]>
Signed-off-by: Marvin Hansen <[email protected]>

Signed-off-by: Marvin Hansen <[email protected]>
Signed-off-by: Marvin Hansen <[email protected]>
Signed-off-by: Marvin Hansen <[email protected]>
@marvin-hansen marvin-hansen merged commit 5e85ee8 into deepcausality-rs:main Aug 20, 2023
6 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant