Skip to content

Verify number validation options#560

Merged
dmdashenkov merged 8 commits intomasterfrom
verify-external-validation-options
Aug 27, 2020
Merged

Verify number validation options#560
dmdashenkov merged 8 commits intomasterfrom
verify-external-validation-options

Conversation

@dmdashenkov
Copy link
Contributor

@dmdashenkov dmdashenkov commented Aug 26, 2020

Starting from now, we will check that (min), (max), and (range) constraints on whole numbers (int32, etc.) are themselves whole numbers.

Also, some leftover debug output is trimmed.

@dmdashenkov dmdashenkov self-assigned this Aug 26, 2020
@codecov
Copy link

codecov bot commented Aug 26, 2020

Codecov Report

Merging #560 into master will increase coverage by 0.00%.
The diff coverage is 64.70%.

@@            Coverage Diff            @@
##             master     #560   +/-   ##
=========================================
  Coverage     73.98%   73.99%           
- Complexity     2969     2975    +6     
=========================================
  Files           506      506           
  Lines         11990    12001   +11     
  Branches        671      674    +3     
=========================================
+ Hits           8871     8880    +9     
+ Misses         2892     2891    -1     
- Partials        227      230    +3     

@dmdashenkov dmdashenkov requested a review from armiol August 27, 2020 07:47
@dmdashenkov
Copy link
Contributor Author

@armiol, PTAL.

Copy link
Collaborator

@armiol armiol left a comment

Choose a reason for hiding this comment

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

@dmdashenkov LGTM. However, please respect #561 in this changeset as well. We need those structural changes for the publishing procedure.

@dmdashenkov dmdashenkov merged commit 4aea052 into master Aug 27, 2020
@dmdashenkov dmdashenkov deleted the verify-external-validation-options branch August 27, 2020 13:48
@dmitrykuzmin dmitrykuzmin mentioned this pull request Sep 8, 2020
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.

External validation constraint must be defined within a numeric type

2 participants