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

YAML 1.2 tab in nb-json #9

Closed
LLyaudet opened this issue Apr 1, 2016 · 1 comment
Closed

YAML 1.2 tab in nb-json #9

LLyaudet opened this issue Apr 1, 2016 · 1 comment

Comments

@LLyaudet
Copy link

@LLyaudet LLyaudet commented Apr 1, 2016

Hi,

In section "5.1 Character Set" Maybe I've misunderstood something but you say explicitly that "JSON quoted scalars cannot [...] contain tabs".
Nevertheless, you list tab "#x9" in rule [2] that defines nb-json. Should-it be renamed "nb-yaml" or should the "#x9" be removed?
If not, can you clarify please?

Best regards,
Laurent Lyaudet

@ingydotnet
Copy link
Member

@ingydotnet ingydotnet commented Jul 31, 2021

Just getting around to open issues here...

I agree that the name nb-json is not great. It was likely meant to mean nb-json-compatibility. I think nb-quoted-char is more appropriate.

I'll attempt to clarify in the upcoming 1.2.1 spec, but the production name change will need to wait until 1.3.

Loading

ingydotnet added a commit that referenced this issue Jul 31, 2021
#9 points out that this section
is confusing. This commit attempts to clarify the section.
ingydotnet added a commit that referenced this issue Jul 31, 2021
#9 points out that this section
is confusing. This commit attempts to clarify the section.
ingydotnet added a commit that referenced this issue Jul 31, 2021
#9 points out that this section
is confusing. This commit attempts to clarify the section.
ingydotnet added a commit that referenced this issue Jul 31, 2021
#9 points out that this section
is confusing. This commit attempts to clarify the section.
ingydotnet added a commit that referenced this issue Jul 31, 2021
#9 points out that this section
is confusing. This commit attempts to clarify the section.
ingydotnet added a commit that referenced this issue Aug 7, 2021
ingydotnet added a commit that referenced this issue Aug 7, 2021
ingydotnet added a commit that referenced this issue Aug 7, 2021
ingydotnet added a commit that referenced this issue Aug 7, 2021
ingydotnet added a commit that referenced this issue Aug 8, 2021
ingydotnet added a commit that referenced this issue Aug 8, 2021
ingydotnet added a commit that referenced this issue Aug 9, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants