Merged
Conversation
Changes: - replace all instances of 'oneOf' with 'anyOf' - flatten 'anyOf' structures with null and array if they aren't too complicated
Backmerge bugfix change
Changes: - add "minLength": 1 restriction to Catalog and Data Service - flatten "anyOf" in Dataset to match the structure/format in Catalog and Data Service - fix typos in property descriptions to align them across the classes
…tion redo changes from "standardize 'keyword' restictions" PR
Relevant classes: Activity, Agent, Catalog, Data Service, Dataset, Document, and Standard Changes: - change 'category' from a single entry to a list in all relevant classes but Dataset (it's already a list here) - replace 'oneOf' with 'anyOf' and 'flatten' in line with PR #113 - fix property descriptions for consistency and to align with changes
Both utilizations have a separate `spatial` field that uses location, and location contains bounding box information. This is redundant, and the GeographicBoundingBox class is not a standard Geospatial representation (GeoJSON, ESRI, etc).
Updates to allow integrated GeoJSON, as well as WKT and GML formats. Removes IRI restriction. Change descriptions to clarify different use cases for different fields.
Move aside outdated files and emphasize the JSON Schema
Replace all 'oneOf' with 'anyOf' and 'flatten' where appropriate
standardize 'category' usage by changing all occurrences to a list
Cleanup/location and geo
remove 'redundant' classes and replace with strings
New Examples
Generate docs from schema files and store them in this repository
…rrent allowances.
zopalmer14
approved these changes
Apr 6, 2026
zopalmer14
left a comment
zopalmer14
left a comment
There was a problem hiding this comment.
Overall, LGTM. I was going relatively quick, so I may have missed some stuff, but I noticed two typos/accidental mistakes and left one note for the record. I also didn't really review the doc_templates and docs .md files.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
These changes are a result of the pull requests approved by the tiger team.
These are to address the following issues:
oneOfcausing problems #90