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

fix(component): removed ivy checks as obsolete #2579

Merged
merged 1 commit into from
Jun 25, 2020

Conversation

BioPhoton
Copy link
Contributor

@BioPhoton BioPhoton commented Jun 14, 2020

Removed unused checks and tests
closes #2575

@ngrxbot
Copy link
Collaborator

ngrxbot commented Jun 14, 2020

Preview docs changes for bc20593 at https://previews.ngrx.io/pr2579-bc205933/

Copy link
Member

@alex-okrushko alex-okrushko left a comment

Choose a reason for hiding this comment

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

#2547 removes isNgZone and adjusts hasZone - and uses it instead.

Could you please adjust the Pr to only remove Ivy-related files?

@brandonroberts brandonroberts added the Needs Cleanup Review changes needed label Jun 17, 2020
@brandonroberts
Copy link
Member

There are merge conflicts that need to be fixed up here also.

@BioPhoton BioPhoton force-pushed the remove-global-this branch from bc1c684 to 9851f60 Compare June 23, 2020 17:36
@BioPhoton BioPhoton force-pushed the remove-global-this branch from 9851f60 to bc20593 Compare June 23, 2020 17:39
@brandonroberts brandonroberts removed the Needs Cleanup Review changes needed label Jun 25, 2020
@brandonroberts brandonroberts merged commit e239950 into ngrx:master Jun 25, 2020
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.

Component: fix globalThis handling
4 participants