all: reenable lint rules after Angular 19 upgrade (connects #9082)#9806
all: reenable lint rules after Angular 19 upgrade (connects #9082)#9806
Conversation
|
All of them are reenabled now except for See https://github.com/angular-eslint/angular-eslint/blob/v18.4.3/packages/eslint-plugin/docs/rules/no-host-metadata-property.md and angular/angular#52561 |
21608b8 to
074fd3d
Compare
|
Note this also upgrades ESLint to v9. typescript-eslint v8 supports ESLint v9 https://typescript-eslint.io/blog/announcing-typescript-eslint-v8 |
|
Figured out it's not working in Node 18 vs 20, but it could also be fixed by changing to the new default config format for ESLint v9. See https://eslint.org/docs/latest/use/migrate-to-9.0.0#-new-default-config-format-eslintconfigjs |
|
@Mutugiii Upgraded the config file for ESLint and it appears to be working with Node 18 now. Keeping the old file because the commented out rules did not make it into the new file. |
|
I'm converting this to a draft because the latest change broke the linting for chatapi. |
Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com> Co-authored-by: paulbert <9203229+paulbert@users.noreply.github.com>
|
Jules came up with a much simpler fix, ready for review now @Mutugiii |



Some lint rules were disabled after the upgrade to Angular 19. Reenabling them here. See https://github.com/open-learning-exchange/planet/pull/9754/changes#diff-6884918dc8291219be508e05e28965b958c734def85324f3b53858ea4702090f for the disabled rules.
Note that for the quotes rule
{ "allowTemplateLiterals": "never" }still allows template literals. It actually throws an error when backticks are used without a template literal or with a one line string.