FR: contextual-interface #826

rafaelss95 · 2021-05-14T22:35:01Z

rafaelss95
May 14, 2021
Collaborator

Currently we have a rule to check if a lifecycle is in the correct context:

contextual-lifecycle

But in reality, there are many other interfaces that I would like to see being checked. Here is the complete list that I managed to put together:

Full list
CanActivate
CanActivateChild
CanDeactivate
CanLoad
Resolve
HttpInterceptor
PipeTransform
ControlValueAccessor
OnChanges
OnInit
DoCheck
AfterContentInit
AfterContentChecked
AfterViewInit
AfterViewChecked
OnDestroy
DoBootstrap

That being said, I'd like to propose a rule to enforce all these interfaces (and probably replace the contextual-lifecycle).
Note that this rule could also accept an optional configuration to let people creation its custom configuration. An example could be like this:

{
  "rules": {
    "@angular-eslint/contextual-interface": [
      "error",
      {
        "OnInitEffects": {
          "decorators": [
            "Injectable"
          ]
        },
        "MyInterface": {
          "decorators": [
            "Component", "Directive"
          ]
        }
      }
    ]
  }
}

Note that this config shouldn't replace the default, but merge.

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

FR: contextual-interface #826

Uh oh!

{{title}}

Uh oh!

Uh oh!

{{editor}}'s edit

{{editor}}'s edit

Uh oh!

Replies: 0 comments

Select a reply

Uh oh!

Uh oh!

FR: contextual-interface #826

Uh oh!

Uh oh!

rafaelss95 May 14, 2021 Collaborator

Replies: 0 comments

rafaelss95
May 14, 2021
Collaborator