-
-
Notifications
You must be signed in to change notification settings - Fork 18
Conversation
Hi @jsoref!, thanks for the Pull Request The pull request title isn't properly formatted. We ask that you update the message to match this format, as we use it to generate changelogs and automate releases.
To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page. Read more about contributing to ESLint here |
Hi @jsoref!, thanks for the Pull Request The pull request title isn't properly formatted. We ask that you update the message to match this format, as we use it to generate changelogs and automate releases.
To Fix: You can fix this problem by clicking 'Edit' next to the pull request title at the top of this page. Read more about contributing to ESLint here |
tools/get-keys-from-ts.js
Outdated
* Finds a TypeScript interfaction declaration. | ||
* Finds a TypeScript interaction declaration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
interfaction is a valid word
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't find anything that indicates it's intended: https://www.google.com/search?sxsrf=APwXEdcQwHkJy68lJ42fE3U3bPNyjsnFlQ:1685414019833&q=%22TypeScript+interfaction%22&nfpr=1&sa=X&ved=2ahUKEwix9M2_gJz_AhUFlYkEHcVWBOoQvgUoAXoECA4QAg&biw=948&bih=1000&dpr=2
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looking more closely, I think the intended word was actually interface
...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes, interface
.
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
Signed-off-by: Josh Soref <2119212+jsoref@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
This PR corrects misspellings identified by the check-spelling action.
The misspellings have been reported at https://github.com/jsoref/eslint-visitor-keys/actions/runs/5116262597#summary-13845436736
The action reports that the changes in this PR would make it happy: https://github.com/jsoref/eslint-visitor-keys/actions/runs/5116262782#summary-13845436773