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

A collection of mostly editorial changes #709

Merged
merged 4 commits into from Jan 17, 2019

Conversation

Projects
None yet
4 participants
@ndw
Copy link
Contributor

ndw commented Jan 16, 2019

I'm about half-way through on my editorial pass. I believe that all of these changes are editorial in the sense that they don't change the technical content of the spec. But a few turned out to be bigger than just typos.

I've tried to avoid reflowing paragraphs and other things to keep the diffs clean(ish). There's also a formatted version here: http://xpspectest.nwalsh.com/editorial/head/xproc/

Diffs are still broken, alas :-(

ndw added some commits Jan 16, 2019

@ndw ndw requested review from gimsieke , eriksiegel and xml-project Jan 16, 2019

@xml-project
Copy link
Contributor

xml-project left a comment

I am not so happy with some part of the terminology like "a step is sending its result" or "verbatim copy of document". I agree with the rules but worry a little bit whether the rule are to close to one possible implementation strategy. May be that is editorial work for the summer.

@ndw

This comment has been minimized.

Copy link
Contributor Author

ndw commented Jan 17, 2019

Suggestions welcome. I'll do a second read through after the first.

@ndw ndw merged commit 043aa00 into xproc:master Jan 17, 2019

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@ndw ndw deleted the ndw:editorial branch Jan 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment