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

Minor DDO service builder improvement #868

Merged
merged 2 commits into from
Jun 7, 2023
Merged

Conversation

mirgee
Copy link
Contributor

@mirgee mirgee commented Jun 2, 2023

Use the typestate pattern to statically ensure service type was set.

@mirgee mirgee force-pushed the refactor/service-builder branch from e1ee705 to 886d6d1 Compare June 2, 2023 13:39
@codecov-commenter
Copy link

codecov-commenter commented Jun 2, 2023

Codecov Report

Merging #868 (4107359) into main (694c9f7) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #868   +/-   ##
=======================================
  Coverage   48.88%   48.88%           
=======================================
  Files         432      432           
  Lines       34324    34324           
  Branches     7609     7609           
=======================================
  Hits        16781    16781           
+ Misses      12267    12266    -1     
- Partials     5276     5277    +1     
Flag Coverage Δ
unittests-aries-vcx 48.85% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

see 2 files with indirect coverage changes

@mirgee mirgee force-pushed the refactor/service-builder branch from 886d6d1 to d5ffa6c Compare June 2, 2023 15:04
@mirgee mirgee force-pushed the refactor/service-changes branch from 6c610e4 to 731a9af Compare June 6, 2023 15:32
@mirgee mirgee force-pushed the refactor/service-builder branch from d5ffa6c to d0279a6 Compare June 6, 2023 15:33
@mirgee mirgee marked this pull request as ready for review June 6, 2023 15:34
@mirgee mirgee force-pushed the refactor/service-changes branch from 731a9af to 7678382 Compare June 7, 2023 05:45
@mirgee mirgee force-pushed the refactor/service-builder branch from d0279a6 to 19d0d81 Compare June 7, 2023 05:46
@mirgee mirgee requested a review from Patrik-Stas June 7, 2023 12:55
Base automatically changed from refactor/service-changes to main June 7, 2023 12:57
mirgee added 2 commits June 7, 2023 15:15
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
Signed-off-by: Miroslav Kovar <miroslav.kovar@absa.africa>
@mirgee mirgee force-pushed the refactor/service-builder branch from 19d0d81 to 4107359 Compare June 7, 2023 13:15
@Patrik-Stas Patrik-Stas merged commit 95e8124 into main Jun 7, 2023
@Patrik-Stas Patrik-Stas deleted the refactor/service-builder branch June 7, 2023 14:24
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.

3 participants