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

feat(component-change-detection): add change detection strategy rule #737

Merged
merged 2 commits into from
Feb 5, 2019

Conversation

mattlewis92
Copy link
Contributor

Please let me know if there's any changes I need to make! 😄

Closes #135

Copy link
Collaborator

@wKoza wKoza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @mattlewis92,

Glad to see you again! I have few comments on your PR, but your code is on the good way.

src/componentChangeDetectionRule.ts Outdated Show resolved Hide resolved
src/componentChangeDetectionRule.ts Outdated Show resolved Hide resolved
src/componentChangeDetectionRule.ts Outdated Show resolved Hide resolved
@mattlewis92 mattlewis92 force-pushed the feat-component-change-detection branch from 40792ac to 86b7be0 Compare February 5, 2019 04:20
@mattlewis92
Copy link
Contributor Author

@wKoza removed the option, LMK what you think! 😄

@wKoza
Copy link
Collaborator

wKoza commented Feb 5, 2019

LGTM ! Thks!

@mgechev mgechev merged commit a23ccde into mgechev:master Feb 5, 2019
@mgechev
Copy link
Owner

mgechev commented Feb 5, 2019

Let's release this once we introduce support for 8.0.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants