-
Notifications
You must be signed in to change notification settings - Fork 22.6k
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
Reword m flag description #37711
Reword m flag description #37711
Conversation
I didn't understand what `m` meant from this page's description, so I reworded it. Hopefully this is more clear.
Preview URLs
(comment last updated: 2025-01-24 18:14:58) |
How do you like this wording? Mostly want to align with |
I think I should share context that led me to submit this PR: I've read this table many times before, but I never really understood what the So whenever I read I now know that The new wording is, What would you think about changing it to Thanks as always! |
I'm okay with that. I originally went with your wording, but it was slightly too long and caused the entire table to grow, but if you agree with it let's leave that concern behind. |
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've applied your suggestion—with some minor tweaks. Thanks!
@Josh-Cena Looks good! I think |
I decided to add that specifically to disambiguate. Otherwise I personally could read it as "match the start and end of each line, instead of matching the entire string." I just don't want someone to send an issue saying it doesn't do what it says. |
Description
Reword m flag description
Motivation
I didn't understand what
m
meant from this page's description, so I reworded it. Hopefully this is more clear.Additional details
Related issues and pull requests