Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat: Fix strict ajv warnings #167

Merged
merged 5 commits into from
Dec 6, 2021
Merged

Conversation

l0b0
Copy link
Contributor

@l0b0 l0b0 commented Nov 17, 2021

See https://ajv.js.org/strict-mode.html.

These are the last warnings in our schemas - see npx ospec 2> >(grep --fixed-strings --invert-match --regex='stac-extensions.github.io' --regex=schemas.stacspec.org >&2) (this command basically removes the warnings for stac-extensions.github.io and schemas.stacspec.org from standard error, showing only the LINZ-related ajv warnings).

MitchellPaff
MitchellPaff previously approved these changes Nov 17, 2021
@l0b0 l0b0 force-pushed the feat/fix-strict-ajv-warnings branch from cbb5a3f to ea3ae04 Compare November 17, 2021 22:06
@l0b0 l0b0 added the automerge kodiak automerge label label Nov 21, 2021
@l0b0 l0b0 force-pushed the feat/fix-strict-ajv-warnings branch 2 times, most recently from 244ab5d to c4995c8 Compare November 22, 2021 21:45
@l0b0 l0b0 marked this pull request as draft November 22, 2021 21:45
@l0b0 l0b0 marked this pull request as ready for review November 22, 2021 21:53
@l0b0 l0b0 force-pushed the feat/fix-strict-ajv-warnings branch from 16d5ef2 to 71cdb55 Compare November 22, 2021 21:54
@l0b0 l0b0 requested review from amfage, MitchellPaff and billgeo and removed request for blacha, billgeo, amfage, MDavidson17 and MitchellPaff November 22, 2021 21:55
@l0b0 l0b0 marked this pull request as draft November 24, 2021 22:10
@l0b0 l0b0 force-pushed the feat/fix-strict-ajv-warnings branch from ab8d690 to c2d30a0 Compare November 24, 2021 23:59
@l0b0 l0b0 requested review from amfage and MitchellPaff November 24, 2021 23:59
@l0b0 l0b0 marked this pull request as ready for review November 24, 2021 23:59
@l0b0 l0b0 requested a review from blacha November 25, 2021 00:01
@l0b0 l0b0 requested a review from MDavidson17 November 25, 2021 00:01
MitchellPaff
MitchellPaff previously approved these changes Nov 29, 2021
@kodiakhq kodiakhq bot removed the automerge kodiak automerge label label Dec 2, 2021
@kodiakhq
Copy link
Contributor

kodiakhq bot commented Dec 2, 2021

This PR currently has a merge conflict. Please resolve this and then re-add the automerge label.

@l0b0 l0b0 added the automerge kodiak automerge label label Dec 6, 2021
@kodiakhq kodiakhq bot merged commit d2f7daa into master Dec 6, 2021
@kodiakhq kodiakhq bot deleted the feat/fix-strict-ajv-warnings branch December 6, 2021 00:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge kodiak automerge label
Development

Successfully merging this pull request may close these issues.

4 participants