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

either "key" or "keys" must be present on all⁽¹⁾ fields #78

Merged
merged 3 commits into from
Dec 9, 2022

Conversation

tyrasd
Copy link
Collaborator

@tyrasd tyrasd commented Dec 9, 2022

This add a check to the schema, making sure that no fields1 without a key or keys property are added.

see openstreetmap/id-tagging-schema#676

Footnotes

  1. exceptions to the rule exist for field types address, wikidata, wikipedia and restrictions

@tyrasd tyrasd added the enhancement New feature or request label Dec 9, 2022
* field types `wikipedia`, `wikidata` and `address` have both `key` and `keys`,
* field type `restrictions` has neither
@tyrasd tyrasd changed the title either "key" or "keys" must be present on all fields either "key" or "keys" must be present on all(*) fields Dec 9, 2022
@tyrasd tyrasd changed the title either "key" or "keys" must be present on all(*) fields either "key" or "keys" must be present on all¹ fields Dec 9, 2022
@tyrasd tyrasd changed the title either "key" or "keys" must be present on all¹ fields either "key" or "keys" must be present on all⁽¹⁾ fields Dec 9, 2022
@tyrasd tyrasd merged commit d2c7b8b into main Dec 9, 2022
@tyrasd tyrasd deleted the schema-required-key-keys branch December 9, 2022 10:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant