Skip to content
This repository has been archived by the owner on Dec 16, 2021. It is now read-only.

feat!: strict data contract schema validation #372

Merged
merged 5 commits into from
Jun 18, 2021
Merged

Conversation

shuplenkov
Copy link
Member

@shuplenkov shuplenkov commented Jun 15, 2021

Issue being fixed or feature implemented

What was done?

Dpp updated to 0.20.0-dev
Ajv updated to 8.6.0

How Has This Been Tested?

Breaking Changes

Contracts must be updated

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

feat: update ajv to 8.6.0
@shuplenkov shuplenkov requested a review from shumkov June 15, 2021 13:24
@shuplenkov shuplenkov added this to the v0.20 milestone Jun 15, 2021
@shumkov shumkov changed the title feat: update dpp to 0.20.0-dev.1 feat: strict data contract schema validation Jun 15, 2021
shumkov
shumkov previously approved these changes Jun 15, 2021
Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

👍

@shumkov shumkov self-requested a review June 15, 2021 14:11
@shumkov shumkov dismissed their stale review June 15, 2021 14:11

dpp is not initialized

Copy link
Member

@shumkov shumkov left a comment

Choose a reason for hiding this comment

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

DPP should be initialized

@shuplenkov shuplenkov changed the title feat: strict data contract schema validation feat!: strict data contract schema validation Jun 18, 2021
@shuplenkov shuplenkov merged commit 33905f0 into v0.20-dev Jun 18, 2021
@shumkov shumkov deleted the update-dpp branch July 22, 2021 12:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants