-
Notifications
You must be signed in to change notification settings - Fork 6
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
is there something like isort for js? #354
Comments
@surchs I believe we can do this by adding the "sort-keys" rule to the eslint rc file, and then by installing/configuring this plugin. |
Cool! I'm in favour of adopting the eslint rule to keep things consistent. I'm less clear about the tool, it looks not very widely adopted / maintained. I could imagine that having the rule enabled and configured to show up in the IDE (and during pre-commit) might be sufficient here. wdyt? |
'sort-keys' definitely works. I tried it out before I commented earlier. I can see it being annoying to the coder to do it manually. I'm alright with it though myself. That said, I don't know if |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 30 days. |
We want to keep our issues up to date and active. This issue hasn't seen any activity in the last 75 days.
|
I notice we sometimes have diffs to re-sort object keys and such things. Is there maybe some tool we can configure to go through the whole codebase once to make the appropriate changes, and to then keep on top of this - e.g. as part of a pre-commit check?
The text was updated successfully, but these errors were encountered: