-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
Update dev dependencies; apply lint #8763
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
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.
Should be good. Will follow up with a non-dev dependency bump
| } | ||
|
|
||
| linkifyIssues(getRepo()!, paragraph); | ||
| }; |
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.
be gone
| 'import/consistent-type-specifier-style': 'off', | ||
| 'import-x/consistent-type-specifier-style': 'off', |
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 don't know what's up with antfu's eslint config and at this point I'm afraid to ask.
|
Please add labels to this PR |
| return [...readmeContent.matchAll(featureRegex)] | ||
| .map(match => extractDataFromMatch(match)) | ||
| .sort((firstFeature, secondFeature) => firstFeature.id.localeCompare(secondFeature.id)); | ||
| .toSorted((firstFeature, secondFeature) => firstFeature.id.localeCompare(secondFeature.id)); |
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.
why use toSorted on a newly created array?
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.
almost all reverse -> toReversed and sort -> toSorted changes seem to be meaningless
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.
you are creating not two, but three arrays here 😭
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.
Such is life sindresorhus/eslint-plugin-unicorn#2735 (comment)
buncha autofixes. Needs review