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.

Sign up for GitHub

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

Jump to bottom

disallow more code points and outright ban certain ones from KDL docu… #353

Merged
merged 1 commit into from
Dec 13, 2023
Merged

disallow more code points and outright ban certain ones from KDL docu… #353

merged 1 commit into from
Dec 13, 2023

Conversation

zkat
Copy link
Member

@zkat zkat commented Dec 11, 2023

…ments altogether

Fixes: #250

@zkat zkat force-pushed the zkat/allow-angles-and-comma branch from bf32dd0 to 9ae43da Compare December 11, 2023 00:37
@zkat zkat force-pushed the zkat/more-code-point-exclusions branch from b6b1b41 to 8cb8466 Compare December 11, 2023 00:38
@zkat zkat added the breaking This can only be done for the next major version of KDL label Dec 11, 2023
@zkat zkat force-pushed the zkat/allow-angles-and-comma branch from 9ae43da to 9c5a061 Compare December 11, 2023 01:49
@zkat zkat force-pushed the zkat/more-code-point-exclusions branch from 8cb8466 to c30cc1c Compare December 11, 2023 01:52
@zkat zkat linked an issue Dec 11, 2023 that may be closed by this pull request
v2.0 additional restricted literal characters #250
Closed
@zkat zkat merged commit ba11ffc into zkat/allow-angles-and-comma Dec 13, 2023
@zkat zkat deleted the zkat/more-code-point-exclusions branch December 13, 2023 04:21
zkat added a commit that referenced this pull request Dec 13, 2023
@zkat
allow ,<> as identifier characters since they no longer need to be re… (
e6356d5 
#352)

* fix some confusion in grammar syntax, and actually specify the syntax itself

Fixes: #345

* allow ,<> as identifier characters since they no longer need to be reserved

* fix typo

* disallow more code points and outright ban certain ones from KDL documents altogether (#353)

Fixes: #250

* `r` prefix is no longer required for raw strings (#354)

Fixes: #337
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
breaking This can only be done for the next major version of KDL
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

v2.0 additional restricted literal characters
1 participant
@zkat