Bump version 0.1.6 - #70
Conversation
There was a problem hiding this comment.
Code Review
This pull request attempts to upgrade the runtime to Node.js 22 and update dependencies. However, there are several inconsistencies: the runtime in extension.yaml and the engine requirement in functions/package.json were downgraded to Node.js 20/>=20 instead of being upgraded to Node.js 22, contradicting the changelog and package-lock.json. Additionally, typescript was incorrectly moved from devDependencies to production dependencies.
|
Thanks for the review @gemini-code-assist! Here is context on why these configurations are set as they are for Firebase Extensions publishing:
|
|
Thanks for the detailed explanation, @huangjeff5. That makes perfect sense. Given the current constraints of the |
No description provided.