Skip to content

Separate validation of the integer type schema from the number type schema #123

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

trusta
Copy link

@trusta trusta commented Feb 20, 2025

Separate the integer type and the number type in validation of all parameters.

@trusta
Copy link
Author

trusta commented Apr 14, 2025

@daveshanley How can I trigger a Codecov Report?

Copy link

codecov bot commented Apr 15, 2025

Codecov Report

Attention: Patch coverage is 57.02479% with 52 lines in your changes missing coverage. Please review.

Project coverage is 94.70%. Comparing base (c322bf8) to head (e93578e).

Files with missing lines Patch % Lines
errors/parameter_errors.go 0.00% 48 Missing ⚠️
parameters/path_parameters.go 90.47% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #123      +/-   ##
==========================================
- Coverage   96.59%   94.70%   -1.90%     
==========================================
  Files          32       32              
  Lines        3378     3493     +115     
==========================================
+ Hits         3263     3308      +45     
- Misses        115      185      +70     
Flag Coverage Δ
unittests 94.70% <57.02%> (-1.90%) ⬇️

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

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant