-
Notifications
You must be signed in to change notification settings - Fork 30
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
Feature: (sort-objects) option for destructuring only #126
Comments
Does |
Unfortunately not; as I said we have many objects in many different places with many different names/no names that we don't want sorted. |
Okay, I know. How about |
Sure, as long as the function receives enough information to differentiate between "regular" objects and destructuring, regardless of what names they may have or the files they're in. |
@azat-io What do you think of this solution? |
What rule do you want to change?
sort-objects
Describe the problem
We have too many objects that we don't want sorted (e.g. XState machines where properties are grouped logically) to make sort-objects with ignore-pattern and/or ESLint overrides feasible. It would be nice to still have object destructuring available, particularly for React components where currently the props' type get sorted but the prop destructuring does not.
Code example
Input:
Expected output:
Additional comments
Not sure if it should be an option to sort-objects or a new/split rule. https://github.com/mskelton/eslint-plugin-sort has separate rules for destructuring-properties and object-properties.
Validations
The text was updated successfully, but these errors were encountered: