-
Notifications
You must be signed in to change notification settings - Fork 244
Closed
Labels
BREAKING CHANGEThis is a breaking change and should only be released as part of a new major versionThis is a breaking change and should only be released as part of a new major versionpackage: eslint-pluginAngular-specific TypeScript rulesAngular-specific TypeScript rules
Milestone
Description
The current DEFAULT_ORDER
does not include host
. I'd suggest adding this. The Angular team has advocated for host
over @HostBinding & @HostListener
. This is mentioned in the v20 release notes.
we removed most guidance related to NgModules and revisited the usage of @HostBinding and @HostListener in favor of the host object within the directive metadata
I'm not particular where it should be added.
angular-eslint/packages/eslint-plugin/src/rules/sort-keys-in-type-decorator.ts
Lines 13 to 29 in d37c7bd
const DEFAULT_ORDER = { | |
Component: [ | |
'selector', | |
'imports', | |
'standalone', | |
'templateUrl', | |
'template', | |
'styleUrl', | |
'styleUrls', | |
'styles', | |
'encapsulation', | |
'changeDetection', | |
], | |
Directive: ['selector', 'standalone'], | |
NgModule: ['declarations', 'imports', 'exports', 'providers', 'bootstrap'], | |
Pipe: ['name', 'standalone'], | |
}; |
Metadata
Metadata
Assignees
Labels
BREAKING CHANGEThis is a breaking change and should only be released as part of a new major versionThis is a breaking change and should only be released as part of a new major versionpackage: eslint-pluginAngular-specific TypeScript rulesAngular-specific TypeScript rules