8000 doc: correct it's vs. its usage by Trott · Pull Request #10098 · nodejs/node · GitHub
  • [go: up one dir, main page]

    Skip to content

    Conversation

    @Trott
    Copy link
    Member
    @Trott Trott commented Dec 2, 2016
    Checklist
    • documentation is changed or added
    • commit message follows commit guidelines
    Affected core subsystem(s)

    doc

    Description of change

    Fix a few instances where it's was being used where the possessive
    its was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.

    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    @Trott Trott added the doc Issues and PRs related to the documentations. label Dec 2, 2016
    jasnell pushed a commit that referenced this pull request Dec 5, 2016
    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    
    PR-URL: #10098
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    @jasnell
    Copy link
    Member
    jasnell commented Dec 5, 2016

    Landed in 6b43c3f

    @jasnell jasnell closed this Dec 5, 2016
    Fishrock123 pushed a commit that referenced this pull request Dec 6, 2016
    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    
    PR-URL: #10098
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    jmdarling pushed a commit to jmdarling/node that referenced this pull request Dec 8, 2016
    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    
    PR-URL: nodejs#10098
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    MylesBorins pushed a commit that referenced this pull request Dec 20, 2016
    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    
    PR-URL: #10098
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    MylesBorins pushed a commit that referenced this pull request Dec 21, 2016
    Fix a few instances where `it's` was being used where the possessive
    `its` was called for. Some additional minor copy-editing of nearby text
    (adding a comma and fixing a comma splice) was also performed.
    
    PR-URL: #10098
    Reviewed-By: Anna Henningsen <anna@addaleax.net>
    Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
    Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
    Reviewed-By: Michaël Zasso <targos@protonmail.com>
    Reviewed-By: James M Snell <jasnell@gmail.com>
    @MylesBorins MylesBorins mentioned this pull request Dec 21, 2016
    @Trott Trott deleted the its branch January 13, 2022 22:44
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Labels

    doc Issues and PRs related to the documentations.

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    7 participants

    0