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

Clarify the base-uri property of standard steps #314

Merged
merged 2 commits into from Jan 1, 2020
Merged

Clarify the base-uri property of standard steps #314

merged 2 commits into from Jan 1, 2020

Conversation

@ndw
Copy link
Collaborator

ndw commented Jan 1, 2020

On reflection, I decided that it would be better to clarify the base URI of standard steps. Here's what I changed:

  1. I made it explicit that p:archive, p:archive-manifest, p:compare, p:count, p:error, p:json-join, p:json-merge, p:pack, p:text-count, p:text-join, p:wrap-sequence, p:www-form-urldecode, and p:www-form-urlencode produce documents with no base URI.
  2. I made it explicit that the base URI(s) of the document(s) produced by p:filter, p:unarchive, p:http-request, p:load, p:xquery, p:xslt reflect the base URIs that one would expect.
  3. I changed p:wrap, p:unwrap to say that they preserve all properties instead of none. I think none was a catagorization error that we never noticed.
  4. I clarified the situation for p:insert a little bit.
ndw added 2 commits Jan 1, 2020
@ndw ndw requested a review from xproc/spec-authors Jan 1, 2020
@ndw ndw merged commit ace8518 into xproc:master Jan 1, 2020
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ndw ndw deleted the ndw:iss-308 branch Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.