-
Notifications
You must be signed in to change notification settings - Fork 0
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
Sense key identifiers should be valid XML IDs #1
Comments
Thanks for this advice. I have changed the separator to colon (:), and intend to leave the issue open for discussion. The XML specification that you cite allows colons in IDs, but discourages it since the main use of colons is to separate name space prefixes. The SKI project aims to support broad interoperability rather than specific formats like XML, so the separator could be anything, as long as it does not appear inside a word. There are other reasons why the current format is not suitable for direct use as XML names: some words start with a number, or include problematic characters like slash and apostrophe. I am not sure that these issues should be handled here, because different applications require different conversions, which are straightworward to implement with very few lines of 'sed' code. |
I think it would be much better if the SKIs are also valid as XML IDs. This means they should fit the following pattern
Source: https://www.w3.org/TR/REC-xml/#NT-Name
This means characters like '#' can't be used and should be escaped. How about ':'?
The text was updated successfully, but these errors were encountered: