-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
Replace double-dash with m-dash #14225
Conversation
Thanks for the review, @yin1999! Since there's additional grammatical changes being made here, I've decided to split the Chinese changes into its own PR so it can be reviewed and merged independently (and probably quicker)! |
37db0e6
to
628023e
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.
Approve for ru
. Thanks 🤝
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.
Looks good for l10n-fr, thanks very much!
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 for ja
.
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 for pt-BR
This pull request has merge conflicts that must be resolved before it can be merged. |
Thanks for everyone's review, we're just waiting on @mdn/yari-content-ko's review now and then this can be merged! |
This pull request has merge conflicts that must be resolved before it can be merged. |
Still haven't heard back from the Korean team -- I'm going to assume that this is alright for Korean, and if it isn't, we can revert it for Korean pages later! |
This PR replaces all instances of a double-dash (--) with an m-dash (—) throughout translated content. This synchronizes formatting with upstream.