chore(deps): update dependency eslint-plugin-perfectionist to v2.10.0 #1617
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR Checklist
status: accepting prs
Overview
Supersedes #1582. This includes the lint fixes.
I'm not sure I think most of change are improvements here. It would more sense to keep existing way for some of them. So maybe it would be better to turn
perfectionist/sort-intersection-types
rule off for now? 🤔 I think it sort named types first and then add{}
part. (Also see azat-io/eslint-plugin-perfectionist#143 and azat-io/eslint-plugin-perfectionist#128 - so there may be new config options later)For example:
NodeWithIdentifierName
It was:
And now it is:
But that aside, this PR fixes the lint errors so we can update the eslint plugin :)