-
Notifications
You must be signed in to change notification settings - Fork 67
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
Formatted String Literals #5123
Conversation
30065a7
to
4cb17a4
Compare
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.
I looked over all the docs changes and made one small suggestion but overall looks good! 👍
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.
parser.peg
at eb58396 is broken for me. Is it broken for you as well?
$ make -C compiler/parser
/Users/noah/brimdata/zed/compiler/parser/deps/bin/pigeon -support-left-recursion -o parser.go parser.peg
format error: filename:15114:53: missing ',' before newline in argument list (and 10 more errors)
make: *** [parser.go] Error 6
You are right! Will fix... |
e8f887c
to
0adda42
Compare
6a7a54b
to
280ebf1
Compare
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.
after fixing those last few typos.
The commit removes the template literals and replaces them with formatted string literals. F-strings have a similar functionality but the expression substitution only happens on strings prefixed with the 'f' character.
The commit removes the template literals and replaces them with formatted string literals. F-strings have a similar functionality but the expression substitution only happens on strings prefixed with the 'f' character.
Fixes #4444