Skip to content

fix(compiler): small improvement to error message handling#63255

Open
Vikash9546 wants to merge 1 commit intomicrosoft:mainfrom
Vikash9546:fix/compiler-parser-comment-typos
Open

fix(compiler): small improvement to error message handling#63255
Vikash9546 wants to merge 1 commit intomicrosoft:mainfrom
Vikash9546:fix/compiler-parser-comment-typos

Conversation

@Vikash9546
Copy link

What problem exists

A small typo/grammar issue in parseErrorForMissingSemicolonAfter's documentation can be a minor distraction when working on parser diagnostics.

Why this change is useful

This PR corrects wording in two nearby comments in src/compiler/parser.ts, making the intent clearer without changing any compiler behavior.

Files modified

  • src/compiler/parser.ts

Tests

  • Not added (comment-only change).
  • Local verification: npm run build.
  • Note: npm test fails locally with EMFILE: too many open files, watch in unittests:: sys:: symlinkWatching:: (environment/FS watcher limit); the change does not affect runtime or test logic.

Made with Cursor

Copilot AI review requested due to automatic review settings March 16, 2026 10:33
@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Mar 16, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR corrects minor spelling/grammar in parser documentation/comments to reduce confusion when working around parseErrorForMissingSemicolonAfter, without changing compiler behavior.

Changes:

  • Fix spelling in JSDoc (“mispelled” → “misspelled”).
  • Fix grammar in an inline comment (“this some kind” → “this is some kind”).

You can also share your feedback on Copilot code review. Take the survey.

@MartinJohns
Copy link
Contributor

From the Pull Request Template:

** Please don't send typo fixes! **
Please don't send a PR solely for the purpose of fixing a typo, unless that typo truly hurts understanding of the text. Each PR represents work for the maintainers, and that work should provide commensurate value.

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

Labels

None yet

Projects

Status: Not started

Development

Successfully merging this pull request may close these issues.

3 participants