-
-
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
Allow delimiters with arbitrary length #820
Conversation
askama_derive/src/config.rs
Outdated
syntax.comment_end, | ||
] { | ||
if s.len() < 2 { | ||
let msg = format!("Delimiters must be at least two characters long: {s:?}"); |
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.
Style nits: capitalized error messages, and maybe just inline msg
here in these branches?
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.
fixed
book/src/configuration.md
Outdated
@@ -105,8 +105,8 @@ The following keys can currently be used to customize template syntax: | |||
* `expr_start`, defaults to `{{` | |||
* `expr_end`, defaults to `}}` | |||
|
|||
Values must be 2 characters long and start delimiters must all start with the same | |||
character. If a key is omitted, the value from the default syntax is used. | |||
Values must be 2 at least characters long. |
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.
Values must be 2 at least characters long. | |
Values must be at least 2 characters long. |
Phrasing
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.
Oops, fixed.
No description provided.