Skip to content
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

doc: remove em dashes #32080

Merged
merged 1 commit into from Mar 6, 2020
Merged

doc: remove em dashes #32080

merged 1 commit into from Mar 6, 2020

Commits on Mar 6, 2020

  1. doc: remove em dashes

    Our documentation uses em dashes inconsistently. They are treated
    inconsistently typographically too. (For example, they are sometimes
    surrounded by spaces and sometimes not.) They are also often confused
    with ordinary hyphens such as in the CHANGELOG, where they are
    inadvertently mixed together in a single list. The difference is
    not obvious in the raw markdown but is very noticeable when rendered,
    appearing to be a typographical error (which it in fact is).
    
    The em dash is never needed. There are always alternatives. Remove em
    dashes entirely.
    
    PR-URL: nodejs#32080
    Reviewed-By: Richard Lau <riclau@uk.ibm.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Trott committed Mar 6, 2020
    Configuration menu
    Copy the full SHA
    b6cd215 View commit details
    Browse the repository at this point in the history