Skip to content

Fix grammar errors in the In-process ConPTY spec - #20513

Open
full-bars wants to merge 1 commit into
microsoft:mainfrom
full-bars:fix-typo-conpty-spec
Open

Fix grammar errors in the In-process ConPTY spec#20513
full-bars wants to merge 1 commit into
microsoft:mainfrom
full-bars:fix-typo-conpty-spec

Conversation

@full-bars

@full-bars full-bars commented Aug 6, 2026

Copy link
Copy Markdown

Summary of the Pull Request

Fixes grammar and spelling errors in the in-process ConPTY spec. The original typo is included in the sweep. The branch is rebased onto the latest main.

References and Relevant Issues

None.

Detailed Description of the Pull Request / Additional comments

Docs-only change, no behavior impact. The fixes cover grammar, spelling, and punctuation only. No technical content was changed.

One wording change is worth noting: "fast pass" is corrected to "fast path" in the ScrollConsoleScreenBuffer TODO. The old wording appears nowhere else in this repository.

Validation Steps Performed

Reviewed the full diff of the document.

PR Checklist

  • Closes #xxx
  • Tests added/passed
  • Documentation updated
    • If checked, please file a pull request on our docs repo and link it here: #xxx
  • Schema updated (if necessary)

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@lhecker Leonard Hecker (lhecker) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The document contains many more grammar errors than just this one and I'm not sure if it's worth fixing any of them. But if we do fix them, it's definitely worth to fix more than just 1 out of 100.

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Aug 6, 2026
@microsoft-github-policy-service microsoft-github-policy-service Bot added No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. and removed No-Recent-Activity This issue/PR is going stale and may be auto-closed without further activity. labels Aug 13, 2026
Fixes grammar and spelling errors across the document. The changes are docs-only and have no behavior impact.
@full-bars
full-bars force-pushed the fix-typo-conpty-spec branch from af802f4 to c8f7ae6 Compare August 16, 2026 10:20
@full-bars full-bars changed the title Fix typo: compatability -> compatibility in ConPTY spec Fix grammar errors in the In-process ConPTY spec Aug 16, 2026
@full-bars

Copy link
Copy Markdown
Author

Thanks for the review. I did a full sweep of the document and fixed the grammar and spelling errors across it, 70 changed lines in total. The changes are limited to grammar, spelling, and punctuation. No technical content was changed. The branch is also rebased onto the latest main.

@microsoft-github-policy-service microsoft-github-policy-service Bot added Needs-Attention The core contributors need to come back around and look at this ASAP. and removed Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something labels Aug 16, 2026
@full-bars

Copy link
Copy Markdown
Author

@microsoft-github-policy-service agree

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs-Attention The core contributors need to come back around and look at this ASAP.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants