search

Samsung / netcoredbg

NetCoreDbg is a managed code debugger with GDB/MI, VSCode DAP and CLI interfaces for CoreCLR.
MIT License
838 stars 103 forks source link

Update README.md #169

Closed TechProofreader closed 2 months ago

TechProofreader commented 6 months ago

Summary: Updated grammar, punctuation, capitalization, organization, structure, hyperlinks, and increased overall readability.

As my username suggests, I enjoy reviewing technical documents for grammatical errors and other related needs across the open source community. I'm currently reviewing NetCoreDbg's documents for any needed revisions or edits, so I figured I would begin with the project's landing page, which I felt could use some updating to increase its overall readability. All of the technical aspects have remained unchanged (actual code, for example), so the only edits in this PR are purely grammatical and organizational in nature. Note: For some lines, the diff shows a removal and exact re-writing of the inline code (not sure why), but none of the inline code has been altered.

If you have any questions, please let me know and I will be respond as soon as possible.

Take Care

TechProofreader commented 5 months ago

Hi Gleb,

Yes, you are correct with your findings. I’m not sure why the process/bin names were updated to match the summary capitalization, but I will correct my PR to include the lowercase spellings so that the PR can be merged.

Take Care

TechProofreader commented 5 months ago

Hi again,

The binary and path names have been corrected.

Take Care

TechProofreader commented 5 months ago

Hey @gbalykov,

I corrected your highlights in my June 4th commit. Please let me know if you have any other concerns. I will continue reviewing the rest of this repo's docs in the meantime.

Take Care

gbalykov commented 2 months ago

Thanks for your contribution, this is part of latest release