-
Notifications
You must be signed in to change notification settings - Fork 28
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
Upgrade @typescript-eslint/* #307
Conversation
4f0c92d
to
5ba8610
Compare
…lets just let the user decide if they need it at all
…lets just let the user decide if they need it at all
@@ -4,8 +4,7 @@ | |||
|
|||
// import type MyComponent from './components/my-component'; | |||
|
|||
// Remove this once entries have been added! 👇 | |||
// eslint-disable-next-line @typescript-eslint/no-empty-interface |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of adding new lint disables here, I've just commented out the whole thing
@@ -4,11 +4,11 @@ | |||
import '@glint/environment-ember-loose'; | |||
import '@glint/environment-ember-template-imports'; | |||
|
|||
declare module '@glint/environment-ember-loose/registry' { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
instead of adding new lint disables here, I've just commented out the whole thing
"@rollup/plugin-babel": "^6.0.4", | ||
"babel-plugin-ember-template-compilation": "^2.2.5", | ||
"concurrently": "^9.0.1", | ||
"ember-source": "^5.4.0", | ||
"ember-template-lint": "^6.0.0",<% if (packageManager === 'npm') { %> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
diverging ember-eslint-parser versions causes eslint-plugin-ember to not be able to work with gjs/gts
No description provided.