-
Notifications
You must be signed in to change notification settings - Fork 978
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
Error: Cannot find module 'p-limit' #5619
Comments
Can confirm, this just broke our GitHub Actions. |
Looks like we let a bit of code with a missing dependency slip in. Working on a patch ASAP, but in the mean time please use the previous version. |
FWIW, if you're using |
Yes. It broke our GitHub Actions.
|
I'm seeing the same issue |
Yep, impacted. I have this in my Github Action: curl -sL https://firebase.tools/ | bash Any suggestions on changing this so that I'm not updating versions each time? |
#5620 |
@dustinlyons If you want to tie to a specific version, as described on firebase.tools:
FWIW, the release job for 11.25.1 is running right now, and we are going to address the hole in our CI that let this slip through. Apologies for the trouble everyone! |
Same issue just accessing through the CLI, glad to see its being worked on |
11.25.1 has been released with a fix for this issue. Apologies everyone! |
Fixes it for me, I appreciate the quick response to the issue!! @joehan |
Thank you for the quick response! |
[REQUIRED] Environment info
firebase-tools: v11.25.0
Platform: macOS
[REQUIRED] Test case
see below
[REQUIRED] Steps to reproduce
Turned out I was logged in - downgrading to v11.24 worked fine.
[REQUIRED] Expected behavior
Not a crash!
[REQUIRED] Actual behavior
The text was updated successfully, but these errors were encountered: