Skip to content

Commit

Permalink
Address remaining comments
Browse files Browse the repository at this point in the history
  • Loading branch information
blva committed Jul 28, 2023
1 parent 021623c commit 5eb5f9d
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 10 deletions.
9 changes: 5 additions & 4 deletions checker/check-request-property-min-updated.go
Original file line number Diff line number Diff line change
Expand Up @@ -64,11 +64,12 @@ func RequestPropertyMinIncreasedCheck(diffReport *diff.Diff, operationsSources *
minDiff.To == nil {
return
}
level := ERR
if propertyDiff.Revision.Value.ReadOnly {
level = INFO
}
if IsIncreasedValue(minDiff) {
level := ERR
if propertyDiff.Revision.Value.ReadOnly {
level = INFO
}

result = append(result, ApiChange{
Id: "request-property-min-increased",
Level: level,
Expand Down
2 changes: 1 addition & 1 deletion checker/check-request-property-min-updated_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ func TestRequestPropertyMinDecreasedCheck(t *testing.T) {
require.Len(t, errs, 1)
require.Equal(t, checker.ApiChange{
Id: "request-property-min-decreased",
Text: "the 'age' request property's min was decreased to '10.00'",
Text: "the 'age' request property's min was decreased from '15.00' to '10.00'",
Level: checker.INFO,
Operation: "POST",
Path: "/pets",
Expand Down
10 changes: 5 additions & 5 deletions checker/localizations/localizations.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

0 comments on commit 5eb5f9d

Please sign in to comment.