Skip to content

remove ANALYZE_FOR_ENTRY_COMPONENTS #982

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

Merged
merged 4 commits into from
Apr 13, 2023
Merged

Conversation

jpike88
Copy link
Contributor

@jpike88 jpike88 commented Mar 31, 2023

fixes #981

@jpike88
Copy link
Contributor Author

jpike88 commented Mar 31, 2023

@wawyed this allows the library to be compatible with angular 16, and I've tested it, seems to work. Pls merge when you get a chance

@jpike88
Copy link
Contributor Author

jpike88 commented Apr 6, 2023

@wawyed would appreciate your timely attention on this, thankyou

@jpike88
Copy link
Contributor Author

jpike88 commented Apr 6, 2023

or are we going to wait till 16 is actually out? this will be a breaking change I guess so maybe best to wait

removed ReflectiveInjector as is deprecated
@wawyed
Copy link
Member

wawyed commented Apr 6, 2023

Will have a look when I get some time. Thank you :)

@jpike88
Copy link
Contributor Author

jpike88 commented Apr 13, 2023

angular 16-rc is out, this is becoming more urgent. I'm already using my fork, but this is going to become a big problem soon

@wawyed
Copy link
Member

wawyed commented Apr 13, 2023

Hi, I fixed the downstream tests in the main branch. You might need to rebase your fork. @jpike88

@jpike88
Copy link
Contributor Author

jpike88 commented Apr 13, 2023

done

@wawyed wawyed merged commit 8cbd06b into ui-router:master Apr 13, 2023
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.

Breaks when building with Angular 16
2 participants