Skip to content

Conversation

@NathanWalker
Copy link
Contributor

  • Just updated packages to latest 👍

@msftclas
Copy link

msftclas commented Jan 15, 2019

CLA assistant check
All CLA requirements met.

@bengry
Copy link
Contributor

bengry commented Jan 15, 2019

@NathanWalker looks good. Please verify that this still works with Angular 7.0.x and 7.1.x apps before we merge this. There should be no issues, but I still prefer that we double check this, given that the compiler updated.

@NathanWalker
Copy link
Contributor Author

NathanWalker commented Jan 16, 2019

@bengry sure thing. I setup this Nx workspace using 7.0.x and built angular-react using the latest here which is included in that workspace in custom packed modules. You can use this repo to test against different Angular version to verify. Given 7.0.x works great with latest builds I would assume 7.1.x would as well:
https://github.com/NathanWalker/ng-7-0-test-with-angular-react-latest

@bengry bengry merged commit 468b97d into microsoft:master Jan 16, 2019
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