-
Notifications
You must be signed in to change notification settings - Fork 211
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
datamodel: implement PSL string literal grammar proposal #2996
Conversation
254281a
to
43f5d2e
Compare
43f5d2e
to
31ced5f
Compare
Simplify pest grammar of string literals We want to tokenize invalid escape sequences and report them as such later in validation. This results in better error messages across the board. closes prisma/prisma#4167
...by the combination of the new postgres default parsing code from the scalar list defaults work and the new string literal code in PSL. closes prisma/prisma#12095
31ced5f
to
be1f1b8
Compare
@@ -1,20 +1,26 @@ | |||
use diagnostics::Diagnostics; | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra line
// References: | ||
// - https://www.ietf.org/rfc/rfc4627.txt | ||
// - https://www.json.org/json-en.html | ||
UNICODE_CONTROL_CHARACTER = _{ '\u{0000}'..'\u{001F}' } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should be called ASCII_CONTROL_CHARACTER
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nevermind, looks good.
Simplify pest grammar of string literals
We want to tokenize invalid escape sequences and report them as such
later in validation.
This results in better error messages across the board.
closes prisma/prisma#4167
and confirm that user issue is closed by the combination of the new postgres default parsing code from the
scalar list defaults work and the new string literal code in PSL.
closes prisma/prisma#12095