Skip to content
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

Catch willDestroy Glimmer component hook in require-super-in-lifecycle-hooks rule #1036

Merged

Conversation

bmish
Copy link
Member

@bmish bmish commented Dec 8, 2020

Fixes #1035.

@bmish bmish added the bug label Dec 8, 2020
@bmish bmish changed the title Catch glimmer component willDestroy hook in require-super-in-life-cycle-hooks rule Catch glimmer component willDestroy hook in require-super-in-life-cycle-hooks rule Dec 8, 2020
@bmish bmish force-pushed the require-super-in-lifecycle-hooks-glimmer branch from 88c5afa to d527ab0 Compare December 8, 2020 02:15
@bmish bmish changed the title Catch glimmer component willDestroy hook in require-super-in-life-cycle-hooks rule Catch willDestroy component hook in require-super-in-life-cycle-hooks rule Dec 8, 2020
@bmish bmish force-pushed the require-super-in-lifecycle-hooks-glimmer branch from d527ab0 to 648a463 Compare December 8, 2020 22:56
@bmish bmish changed the title Catch willDestroy component hook in require-super-in-life-cycle-hooks rule Catch willDestroy Glimmer component hook in require-super-in-life-cycle-hooks rule Dec 11, 2020
@bmish bmish force-pushed the require-super-in-lifecycle-hooks-glimmer branch from 648a463 to 96afd81 Compare December 11, 2020 00:53
@bmish bmish merged commit efdbd1d into ember-cli:master Dec 11, 2020
@bmish bmish changed the title Catch willDestroy Glimmer component hook in require-super-in-life-cycle-hooks rule Catch willDestroy Glimmer component hook in require-super-in-lifecycle-hooks rule Dec 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[require-super-in-lifecycle-hooks] False negative with Glimmer components?
1 participant