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
Fix bug where ESM module load fails on windows. #3692
Conversation
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.
One phrasing nit in CHANGELOG
Co-authored-by: joehan <joehanley@google.com>
Is writing them as ESM explained in docs? |
@web2033 Nothing yet for Firebase Functions, but here's an article for Google Cloud Functions https://medium.com/google-cloud/es-modules-in-cloud-functions-f5be1676c8b5. |
Do I need to add "@google-cloud/functions-framework": "^1.9.0" explicitly? |
@web2033 I will write a Medium article for Firebase Functions on Node 14 with ESM Modules within the next week. |
@fouchekeagan When writing your a blog, you might want to see this: microsoft/vscode#131837. I'd like to see how you set up your per-folder vscode settings. (Unless you don't use typescript to write your functions, then this is totally irrelevant. ) |
This is the minimal 'Hello World' example I came up with (Node 16 and ESM) If I remove |
@web2033 I was able to deploy my function packaged as an ES module without including the Maybe GCF team is slowing rolling out new version of the buildpack that includes the latest version of the |
Node 16? |
@web2033 Oh interesting. Deploying Node 16 function fails for me as well. Node 14 works. |
Do we need an issue to track it? In which package? |
@web2033 The change needs to happen on Google Cloud Functions backend team. I've let them know - will ping this thread when the change is rolled out. If you prefer following an issue, feel free to open one in firebase-functions or open a public one is GCF. https://issuetracker.google.com/issues?q=componentid:187195%20status:open# |
Patch allows Windows users to deploy Firebase Functions packaged as ES module. We apply the same fix we made for Windows ES module support in the Functions Emulator (firebase#3574) to the triggerParser script. Fixes firebase#3689
Patch allows Windows users to deploy Firebase Functions packaged as ES module.
We apply the same fix we made for Windows ES module support in the Functions Emulator (#3574) to the triggerParser script.
Fixes #3689