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

(#32) webApi: add classlib and tests #35

Merged
merged 14 commits into from
Sep 22, 2024
Merged

(#32) webApi: add classlib and tests #35

merged 14 commits into from
Sep 22, 2024

Conversation

SaintAngeLs
Copy link
Contributor

(#32)

Copy link

codecov bot commented Sep 22, 2024

Codecov Report

Attention: Patch coverage is 73.19588% with 26 lines in your changes missing coverage. Please review.

@@            Coverage Diff             @@
##             main      #35      +/-   ##
==========================================
- Coverage   59.94%   50.10%   -9.84%     
==========================================
  Files          45       49       +4     
  Lines        1036     1407     +371     
  Branches      123      168      +45     
==========================================
+ Hits          621      705      +84     
- Misses        356      633     +277     
- Partials       59       69      +10     
Flag Coverage Δ
Paralax 86.86% <ø> (ø)
Paralax.CQRS.Commands 88.88% <ø> (ø)
Paralax.CQRS.EventSourcing 78.12% <ø> (ø)
Paralax.CQRS.Events ?
Paralax.CQRS.Logging 71.32% <ø> (ø)
Paralax.CQRS.Queries 52.43% <ø> (ø)
Paralax.Docs.Scalar 73.19% <73.19%> (?)
Paralax.HTTP ?
Paralax.Logging 57.93% <ø> (ø)
Paralax.Security 58.41% <ø> (ø)
Paralax.WebApi 26.74% <ø> (?)

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

Components Coverage Δ
Paralax Core 86.86% <ø> (ø)
Paralax CQRS Commands 88.88% <ø> (ø)
Paralax CQRS Queries 52.43% <ø> (ø)
Paralax HTTP ∅ <ø> (∅)
Paralax Security 58.41% <ø> (ø)

Copy link
Contributor

@DevITSharpPRO DevITSharpPRO left a comment

Choose a reason for hiding this comment

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

LGFM!

@SaintAngeLs SaintAngeLs merged commit 2c75191 into main Sep 22, 2024
10 checks passed
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.

2 participants