Skip to content
This repository has been archived by the owner on Jul 4, 2024. It is now read-only.

Update json schema validator library #301

Merged
merged 3 commits into from
Sep 13, 2019

Conversation

dbadura
Copy link
Contributor

@dbadura dbadura commented Sep 13, 2019

Description

Changes proposed in this pull request:

  • update the jsonschemavalidator library with newest fix PR
  • add test for invalid schema

Related issue(s)

See #256

@dbadura dbadura added the kind/feature Categorizes issue or PR as related to a new feature. label Sep 13, 2019
@dbadura dbadura changed the title update library Update jsonschenavalidator library Sep 13, 2019
@dbadura dbadura changed the title Update jsonschenavalidator library Update json schema validator library Sep 13, 2019
@dbadura dbadura marked this pull request as ready for review September 13, 2019 07:56
@@ -187,3 +187,14 @@ func TestValidator_ValidateRaw(t *testing.T) {
})
}
}

// Test for this issue https://github.com/kyma-incubator/compass/issues/256
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove the comment

@@ -21,7 +21,8 @@ required = [

[[constraint]]
name = "github.com/xeipuuv/gojsonschema"
version = "1.1.0"
# we fixed this library and this is the master commit with our fix
Copy link
Contributor

Choose a reason for hiding this comment

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

revision with fix for https://github.com/kyma-incubator/compass/issues/256

@pkosiec pkosiec self-assigned this Sep 13, 2019
@@ -21,7 +21,8 @@ required = [

[[constraint]]
name = "github.com/xeipuuv/gojsonschema"
version = "1.1.0"
# we fixed this library and this is the master commit with our fix
revision = "6a016cf16650f6be7151db7c3be6fc90a2c4d9b1"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
revision = "6a016cf16650f6be7151db7c3be6fc90a2c4d9b1"
revision = "6a016cf16650f6be7151db7c3be6fc90a2c4d9b1"

@dbadura dbadura merged commit 0861d99 into kyma-incubator:master Sep 13, 2019
@dbadura dbadura deleted the update-schema-validator-lib branch September 13, 2019 08:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/feature Categorizes issue or PR as related to a new feature.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants