Skip to content

docs: fix typos in Herebyfile.mjs and eslint rule comment#63324

Closed
ayushshukla1807 wants to merge 1 commit intomicrosoft:mainfrom
ayushshukla1807:fix-typos
Closed

docs: fix typos in Herebyfile.mjs and eslint rule comment#63324
ayushshukla1807 wants to merge 1 commit intomicrosoft:mainfrom
ayushshukla1807:fix-typos

Conversation

@ayushshukla1807
Copy link
Copy Markdown

Fix minor typos found via codespell:\n- 'miscelaneous' → 'miscellaneous' in task descriptions (Herebyfile.mjs)\n- 'arugments' → 'arguments' in a comment (scripts/eslint/rules/argument-trivia.cjs)

Copilot AI review requested due to automatic review settings March 30, 2026 17:40
@github-project-automation github-project-automation bot moved this to Not started in PR Backlog Mar 30, 2026
@jakebailey
Copy link
Copy Markdown
Member

Copy link
Copy Markdown
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 fixes minor spelling typos discovered by codespell in build task descriptions and an eslint rule comment, improving readability without changing behavior.

Changes:

  • Corrects “arugments” → “arguments” in argument-trivia rule comment.
  • Corrects “miscelaneous” → “miscellaneous” in Hereby task descriptions (other-outputs and watch-other-outputs).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
scripts/eslint/rules/argument-trivia.cjs Fixes a typo in an explanatory comment around parameter-name handling.
Herebyfile.mjs Fixes a typo in task description strings for output-building tasks.

@MartinJohns
Copy link
Copy Markdown
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.

@ayushshukla1807
Copy link
Copy Markdown
Author

ayushshukla1807 commented Mar 30, 2026 via email

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

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants