Skip to content

Reword part of Req 27.#1020

Merged
pvretano merged 2 commits intoopengeospatial:masterfrom
pvretano:profile
Sep 29, 2025
Merged

Reword part of Req 27.#1020
pvretano merged 2 commits intoopengeospatial:masterfrom
pvretano:profile

Conversation

@pvretano
Copy link
Copy Markdown
Contributor

The current wording of req 27 is leading people to think that profile URIs need to be registered in the OGC Profile Registry. I've remove the example part for the second part of the requirement and added a note after the requirement that more fully describes the intent ... I hope.

@pvretano pvretano requested a review from cportele September 29, 2025 15:50
@pvretano pvretano moved this from In progress to Review in progress in Features Part 5: Schemas Sep 29, 2025
Copy link
Copy Markdown
Member

@cportele cportele left a comment

Choose a reason for hiding this comment

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

LGTM. I do not understand how an example can be understood as a constraint on the value, but it is clearer to move the example outside of the requirement itself.

@github-project-automation github-project-automation bot moved this from Review in progress to Reviewer approved in Features Part 5: Schemas Sep 29, 2025
@pvretano pvretano merged commit 76debd8 into opengeospatial:master Sep 29, 2025
@github-project-automation github-project-automation bot moved this from Reviewer approved to Done in Features Part 5: Schemas Sep 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants