chore(opentelemetry-context-zone-peer-dep): support zone.js ^v0.13.0 #4320
Conversation
update deps to support angular 16 closes issue open-telemetry#4245 Signed-off-by: rbrennan <4884521+rodgerbrennan@users.noreply.github.com>
|
|
pichlermarc
changed the title
This reverts commit 624df4e.
rodgerbrennan
force-pushed
the
fix/zonejs-version
branch
from
464a72c
to
ea1e5c8
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #4320 +/- ##
=======================================
Coverage 92.20% 92.20%
=======================================
Files 336 336
Lines 9512 9512
Branches 2016 2016
=======================================
Hits 8771 8771
Misses 741 741 |
There was a problem hiding this comment.
FYI @rodgerbrennan I'm not sure that you're aware, but the browser tests for this PR seem to be failing - you can invoke them locally by running npm run compile folllowed by npm run test:browser.
We'll need the tests need to pass before we can merge this PR to ensure we don't introduce a regression for our users.
|
@pichlermarc I believe my latest commit fixes the browser tests |
| @@ -14,7 +14,7 @@ | |||
| * limitations under the License. | |||
| */ | |||
|
|
|||
| import 'zone.js'; | |||
| import 'zone.js/dist/zone'; | |||
There was a problem hiding this comment.
I believe we should enable transpiling the zone.js with webpack if the default entry of zone.js contains advanced syntax.
This can be done in a follow-up PR.
| import 'zone.js/dist/zone'; | |
| // Default 'zone.js' contains advanced syntax that can not be consumed by the test webpack. | |
| // Load the es5 target bundle instead. | |
| import 'zone.js/dist/zone'; |
|
@rodgerbrennan Still requires an entry in |
I added a note in the changelog under unreleased, house(internal) |
update deps to support angular 16
Fixes #4245