-
Notifications
You must be signed in to change notification settings - Fork 147
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
Monospace variable names in draft-variants. #738
base: main
Are you sure you want to change the base?
Conversation
We've tried this syntax in the past, but the "`" marks get into the .txt, and then we have to work with the RFC Editor to remove them before publication. |
I see But I won't be mad if you don't want this change. :) |
Personally I was OK with it, but we got a lot of negative feedback. |
As @jyasskin says, backticks turn into double quotes with no issue. Some other specs in this repo used that with reasonable outcomes. You just have to be sure that you wanted those quotes. |
Yes. I'm saying that my experience is that the RFC Editor and others don't like to see variable names surrounded by "`". |
but they won't see the backticks. Are we speaking past each other? |
Ah - so are you saying that something in between .md and .txt (kramdown-rfc2629 or xml2rfc) now produces a double quote instead of a backtick? When / where did that happen? |
In my experience it's always done that, at least when I build my .txt files locally. (Where "always" means "for at least a couple of years".) For example, search these documents for "n00" -- in the HTML it renders as |
kramdown-rfc2629 turns backticks into There is some risk to using typography in RFCs: https://mozphab-ietf.devsvcdev.mozaws.net/D4234#inline-7837 shows CDDL describing typewriter text, while the reviewer saw quotes. |
OK. I may have been confusing "`" with something else. The only issue I see here is that lines like this:
will become in
... so there won't be any discriminator between a variable reference and a literal. That doesn't seem good. Right now we depend on the uniqueness of the term to identify variables, reserving double-quotes for literals. I agree that the HTML rendering is MUCH better, but sadly that still isn't the authoritative version. |
The discussion is useful, but I wouldn't go there for specs that are planned to be published anytime soon. |
Can we align on this for new spec? I think it improves readability... |
This made it easier for me to read some of the algorithms, but feel free to reject it if it doesn't help you.