Repository navigation
Conversation
|
The change is ready and locally validated, but the Test and publish workflow is currently marked action_required before any jobs start because this fork needs maintainer approval to run Actions. Could a maintainer approve the workflow run when convenient? Thank you. |
cb924f9 to
18009c1
Compare
18009c1 to
1ddf87d
Compare
Have you tried the "actions" tab at your forked repository? I believe you can run those workflows over there if you want to try out the "test and publish" workflow :) |
|
Hi @EthanSK, thanks for this, and sorry for the long silence. I am the new maintainer, and this PR was triaged the day it opened but then held while we got Here's what I found: What your PR already gets right
What changed since you submitted this PR
Where this goes nextI cannot merge any Angular 22 change into So two ways forward, and I am fine with either:
Let me know which you prefer. If I do not hear back by October 14, I'll go ahead with the second option and say so here. Either way, thank you for getting this started months before I could. |
Summary
ComponentFactoryResolverdependency in modular and compat Analytics screen tracking withreflectComponentTypeFixes #3696
Compatibility notes
ComponentFactoryResolverchanges the constructor signature of bothScreenTrackingServiceimplementations and the internalɵscreenViewEventsignature; Angular DI consumers are unaffected, while callers that construct these classes directly must remove that argumentsrc/schematics/package.jsonis intentional: the distributed package is ESM, while the schematics are compiled as CommonJS and need the nested package marker to load throughng addandng updateVerification
npm cinpm run lintnpm run buildnpm run build:jasmine && npm run test:node-esm(226 specs)22.0.0-rc.0tarball in a fresh strict Angular 22 consumer and completed a production build usingprovideAnalytics,ScreenTrackingService, andUserTrackingServiceThe full emulator-backed browser matrix is left to this repository's GitHub Actions run because a shared Firebase Storage emulator is already active locally.