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

Editorial: Use \n instead of \r\n in XML documents #645

Closed
ChristianGruen opened this issue Jul 26, 2023 · 3 comments
Closed

Editorial: Use \n instead of \r\n in XML documents #645

ChristianGruen opened this issue Jul 26, 2023 · 3 comments
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. Serialization An issue related to the Serialization spec

Comments

@ChristianGruen
Copy link
Contributor

If we use a consistent newline encoding in all XML documents, it will be easier to perform cleanups and to edit documents in the GitHub frontend. Most documents already use Unix-style newlines. If I am correct, only three documents need to be updated:

specifications/xslt-xquery-serialization-40/src/errors.xml
specifications/xslt-xquery-serialization-40/src/ns-xslt-xquery-serialization.xml
specifications/xslt-xquery-serialization-40/src/xslt-xquery-serialization.xml
@ChristianGruen ChristianGruen added Editorial Minor typos, wording clarifications, example fixes, etc. Serialization An issue related to the Serialization spec labels Jul 26, 2023
@ChristianGruen ChristianGruen self-assigned this Jul 26, 2023
@michaelhkay
Copy link
Contributor

There's always a danger, of course, that some future contributor will use an editor that turns the newlines back into Windows format.

@ChristianGruen
Copy link
Contributor Author

ChristianGruen commented Jul 28, 2023

There's always a danger, of course, that some future contributor will use an editor that turns the newlines back into Windows format.

True. I think we should reject such changes, at least as long we use GitHub.

@ndw
Copy link
Contributor

ndw commented Jul 30, 2023

Better than that, we can tell git to fix the line endings in comments. I'll see about getting that setup. It's Balisage next week, so I may be a little...distracted.

ChristianGruen added a commit to ChristianGruen/qtspecs that referenced this issue Sep 14, 2023
@ChristianGruen ChristianGruen added the PR Pending A PR has been raised to resolve this issue label Sep 14, 2023
@ChristianGruen ChristianGruen removed their assignment Mar 19, 2024
@ChristianGruen ChristianGruen removed the PR Pending A PR has been raised to resolve this issue label Mar 26, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Editorial Minor typos, wording clarifications, example fixes, etc. Serialization An issue related to the Serialization spec
Projects
None yet
Development

No branches or pull requests

3 participants