Skip to content

Remove erroneous use of "default base IRI" #1324

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

Closed
wants to merge 1 commit into from

Conversation

handrews
Copy link
Contributor

@handrews handrews commented Oct 12, 2022

Whatever else happens with #1299 and #1322, this wording is incorrect. I have filed this draft PR because it seems like the wording may be contributing to the confusion in those issues, so I am emphasizing that this wording error is orthogonal. #1299 in particular should be read as if this change has already been made.

This should just say "base IRI", as a "default base IRI" is a specific kind of base IRI that does not make sense in the context of this paragraph.

The term is used correctly several paragraphs down where there is a RECOMMENDATION that if a default base IRI is defined (which is not in any way required) that it be documented.

I'm marking it as a draft b/c it will likely be superseded by other changes from #1299 and/or #1322. I'm just posting it for clarity.

This should just say "base IRI", as a "default base IRI" is a
specific kind of base IRI that does not make sense in the context
of this paragraph.

The term is used correctly several paragraphs down where there is
a RECOMMENDATION that if a default base IRI is defined (which is
not in any way required) that it be documented.
@handrews
Copy link
Contributor Author

This has now served its purpose and does not need to remain open.

@handrews handrews closed this Oct 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant