-
Notifications
You must be signed in to change notification settings - Fork 5
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
Make inline comments configurable #14
Comments
This also seems relevant to newsdev/archieml-js#5, as well as an issue I came across recently in writing cross-parser tests: how to include JSON as part of an AML document. Because all text inside single brackets is stripped out, that makes it difficult to store JSON that contains an array as a value:
Will get converted to The most simple solution would be to have a setting that toggles this on and off. A more complex solution for parsers would be making it adjustable. |
Toggling off would solve our problem! For this type of content we really don't need inline comments (and if we really did we do it with our templating language)... Adam |
I took a stab at implementing this in I'm inclined to go the toggle route over configuring the comment syntax, since inline comments do not alter the resulting document structure, making it trivial to pass values through a custom regex as needed post-parsing. |
This should work perfectly but we'll test it and let you know. Thank you! |
Copying over from @aplayford in newsdev/archieml-js#8:
The text was updated successfully, but these errors were encountered: