-
-
Notifications
You must be signed in to change notification settings - Fork 516
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
update strict-event-emitter to at least v0.3.x to avoid implicit dependency on Node.js type definitions. #1519
update strict-event-emitter to at least v0.3.x to avoid implicit dependency on Node.js type definitions. #1519
Comments
Hi @kettanaito I think the error would be resolved when updating I'm happy to give this a try if you agree. |
Hey, @chrisguttandin! Thanks for raising this. If MSW depends on this package directly then let's bump it. You won't be able to update the Interceptors because the newest version already ships with the Fetch API support so its being updated as a part of the said feature in MSW. People can use package resolutions for now to avoid this issue, I hope. |
I can use some help with the strict-event-emitter update for the Interceptors though mswjs/interceptors#324 |
I created a fresh fork of ➤ YN0066: │ typescript@patch:typescript@npm%3A4.5.4#builtin<compat/typescript>::version=4.5.4&hash=8133ad: Cannot apply hunk #2 (set enableInlineHunks for details)
➤ YN0066: │ typescript@patch:typescript@npm%3A4.9.3#builtin<compat/typescript>::version=4.9.3&hash=8133ad: Cannot apply hunk #2 (set enableInlineHunks for details) Does that somehow look familiar to you? |
@chrisguttandin, the first time I see that. Maybe check your Node version? I develop on 16.18.0. |
I uninstalled yarn itself and re-installed it again. Now I'm on v1.22.19 of yarn. I'm also using Node.js v16. Everything works as expected now. I can run I've created a PR to update over |
* chore(deps): bump json5 from 0.2.6 to 0.4.3 (#1519) * fix: replace StrictEventEmitter with Emitter
Released: v0.49.3 🎉This has been released in v0.49.3! Make sure to always update to the latest version ( Predictable release automation by @ossjs/release. |
Hi, this latest bugfix release suddenly broke my Ember.js build. |
Thanks for reporting this, @BoussonKarel. Looks like we need to compile |
Will be released as |
I looked into it further yesterday at it seemed like the issue is also ember-auto-import. Class private methods are supposed to be enabled, but apparently they're not. |
The strict-event-emitter update did fix it, thanks! |
I'm facing the same issue with the 1.2.1 on an Angular Project. browser.ts:
I added Any idea what's going on ? |
Prerequisites
Environment check
msw
versionBrowsers
Firefox
Reproduction repository
https://github.com/chrisguttandin/msw-test
Reproduction steps
Run the following commands:
Open
index.ts
and add the following content:Compile the project:
Current behavior
Expected behavior
I would expect it to compile.
The text was updated successfully, but these errors were encountered: