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

Issue 873 #921

Merged
merged 8 commits into from Dec 2, 2019
Merged

Issue 873 #921

merged 8 commits into from Dec 2, 2019

Conversation

@xatapult
Copy link
Contributor

xatapult commented Dec 2, 2019

This was a final cleanup thing. What I did:

  • Changed 's into typographical apostrophes
  • Performed a full spell check on the document (and fixed several misspellings)
  • Removed the single surviving editorial note (on JSON documents in the pipeline, consensus was reached on this)
  • Removed all surviving older text in comments
  • reworked the example in p:urify to a different layout because it looked truly awful (no text changes)
  • Changed the example for p:version-available() to 3.0 (was 1.0)
@xatapult xatapult requested a review from xproc/spec-authors as a code owner Dec 2, 2019
@xatapult xatapult merged commit d4ebd1f into xproc:master Dec 2, 2019
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@xatapult xatapult deleted the xatapult:issue-873 branch Dec 2, 2019
@xml-project

This comment has been minimized.

Copy link
Contributor

xml-project commented Dec 2, 2019

Hurray! Thank you @xatapult

@xatapult

This comment has been minimized.

Copy link
Contributor Author

xatapult commented Dec 2, 2019

Yup. What a milestone!

@ndw

This comment has been minimized.

Copy link
Contributor

ndw commented Dec 2, 2019

Fabulous! Thank you Erik!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.