-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
fix: make type-imports autofixable & general project cleanup #1217
Conversation
🦋 Changeset detectedLatest commit: 64daf93 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Running Lighthouse audit... |
we currently use is it worth making that consistent? |
probably - this lint rule is gonna prefer the inline one but it doesn't complain about the other way. maybe there is a way to do that 🤔 |
Hey t3-oss/translators! This PR contains changes to your language. Please review the changes ❤️. |
Done using this rule: https://github.com/import-js/eslint-plugin-import/blob/main/docs/rules/consistent-type-specifier-style.md |
Closes #1173
Closes #1163 (disabled cache job - not optimal but not the end of the world, can make a separate package if we really want it cached)
Closes #1182
Closes #1181
✅ Checklist
Changelog
Ended up being a bit of everything pr 😮💨
[Short description of what has changed]
CleanShot.2023-02-18.at.22.22.54.mp4
Screenshots
[Screenshots]
💯