-
Notifications
You must be signed in to change notification settings - Fork 35
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
Feature/firebase experiments #731
Feature/firebase experiments #731
Conversation
…rt webframeworks)
- Switched around the ordering of commands to avoid permission denied issues at filesystem level.
@loicmathieu There is also some workaround code in this PR for a bug I ran into (which mostly triggers when using the experiments, hence I added it to this PR). Filed a bug (#732) to remove the logic once the upstream bug is fixed. |
…mmands can run when the dev service is shutdown.
@loicmathieu Have some time to get this merged? Besides the feature, this PR also contains a few fixes for issues I ran in to while working with the extension quite a bit (i.e. really ironing out the corner cases). Can split the PR into multiple if that eases reviewing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks
Firebase has a way to enable experimental features. This is used e.g. when combining next.js with Firebase. The new option allows these experiments to be used.