Skip to content

Spelling - #325

Open
jsoref wants to merge 11 commits into
jasongin:masterfrom
jsoref:spelling
Open

jsoref wants to merge 11 commits into
jasongin:masterfrom
jsoref:spelling

Conversation

@jsoref

@jsoref jsoref commented Sep 20, 2026

Copy link
Copy Markdown

I've manually selected each correction and reviewed the changes.
The work was developed here: https://github.com/check-spelling-sandbox/nvs/actions/ using my tooling to identify errors -- the tooling doesn't provide corrections for typos (but does have hints for some grammatical errors, including the ones fixed here).

Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Comment thread lib/addRemove.js
// The retry with backoff is only applicable to Windows.
throw e;
} else if ((e.code === 'EACCS' || e.code === 'EPERM') && Date.now() < backoffUntil) {
} else if ((e.code === 'EACCES' || e.code === 'EPERM') && Date.now() < backoffUntil) {

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

This is important. Afaik, there is no EACCS error anywhere. Whereas EACCES is an error code produced on systems.

Conceptually, all the other corrections have no real impact on code flow and are essentially cosmetic.

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.

1 participant