-
Notifications
You must be signed in to change notification settings - Fork 370
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
revsets.md: clarify string patterns, and file patterns #2484
Conversation
@martinvonz or @yuja, please check for correctness. I had to experiment a bit to get what I think are the correct rules. In particular, I can't think of a situation where quotes are not optional, but maybe there is some or we want to deprecate it? (It's awfully convenient to do |
9453984
to
984520a
Compare
e588189
to
fccb77e
Compare
I seem to have gotten it wrong, it's ~fixed now. I can also postpone documenting the current |
2f442e6
to
8057f0e
Compare
The indentation change is not very pretty, but making it more consistent would require some trial and error with what both MkDocs and GitHub accept. |
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.
LGTM, but Yuya knows this best
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.
lgtm, thanks.
58532c9
to
9c1e294
Compare
I had to increase indent on two list items for multiple paragraphs to work in MkDocs.
Thanks, Yuya and Martin! |
I had to increase indent on two list items for multiple paragraphs to work in
MkDocs.
Checklist
If applicable:
CHANGELOG.md