Skip to content

Conversation

kvakil
Copy link
Contributor

@kvakil kvakil commented Jul 21, 2022

This diff includes the following changes:

  • Copyediting: grammar, spelling, remove redundant text.
    • Particularly I felt the "Debugging" section was repeated too many
      times; I removed it. Most of the other changes should be
      straightforward.
  • Some additional detail about the trace gc logs.

This diff includes the following changes:

* Copyediting: grammar, spelling, remove redundant text.
    * Particularly I felt the "Debugging" section was repeated too many
      times; I removed it. Most of the other changes should be
      straightforward.
* Some additional detail about the trace gc logs.
@kvakil kvakil force-pushed the copyedit-diagnostics-memory branch from 1adae09 to 351492a Compare July 21, 2022 21:11
@SEWeiTung SEWeiTung requested a review from RafaelGSS July 22, 2022 02:36
Copy link
Member

@RafaelGSS RafaelGSS left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thank you!

@RafaelGSS RafaelGSS merged commit 2718cac into nodejs:main Jul 22, 2022
SEWeiTung pushed a commit that referenced this pull request Jul 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants