-
Notifications
You must be signed in to change notification settings - Fork 647
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
ESLint rule and have Prettier handle import type { Type }
Pattern
#2129
Comments
Hey! Is this issue free to pick up? |
Sure 😄 |
To give an update here: the change that allows to enforce such pattern was just merged. I'll update the configuration in the repository when the release with this PR is published. |
It seems like we also need to this rule to be released to be able to enforce |
Just following up on this, are you still working on this @mrkldshv ? |
Sure! From what I understood we'd need release of https://github.com/import-js/eslint-plugin-import to make it work. The old version doesn't have |
Closing this issue since we've configured Prettier to do this |
@penalosa consistency, we started with the
import type { FetchError } from "../cfetch";
and would like to keep it consistent, we should at it as a ESLint rule and have Prettier handle it if it can.Originally posted by @JacobMGEvans in #2064 (comment)
Cc: @penalosa
The text was updated successfully, but these errors were encountered: