-
Notifications
You must be signed in to change notification settings - Fork 48
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
[BUG] Miniflare testing not working due to chalk #563
Comments
Hey I am running into this as well. Did you find a workaround? |
Unfortunately not. We are using lambda by now so the problem isn't ours anymore. |
Hi folks! Got a probable fix over at #781 (comment).
I can't reproduce with Miniflare, so will wait for verification here. |
@jpwilliams in this particular instance, it isn't that chalk is missing but |
I was able to get it to work! :) First I had to upgrade my dependencies from:
to
then following the instructions from this change: I had to add this to my
note: @jpwilliams i did NOT need your chalk changes to fix this, i think that was a symptom of a bundling issue vs. an inngest problem, but you should probably play it safe and not expect colors to be supported. pinging cloudflare/workers-sdk#7120 since cloudflare/workers-sdk#7810 fixed that for me |
Describe the bug
When running a test with mini flare (Cloudflare Worker local dev env) and Inngestjs, it always crashes with
The reason is that for some reason,
chalk.magenta
isundefined
. Is there any way we could change it to return a default prefix (without chalk) if its not there?The text was updated successfully, but these errors were encountered: