Skip to content
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

chore: update dependencies for firebase functions template #7809

Merged
merged 8 commits into from
Oct 16, 2024

Conversation

emwp
Copy link
Contributor

@emwp emwp commented Oct 8, 2024

Description

Updates the firebase-functions and firebase-admin dependencies for the firebase functions init template.
fixes #7385

Scenarios Tested

Initializing a new project with init functions and deploy commands:

  • It inits and builds without problems
  • Is able to be deployed
  • Outdated packages warning no longer present

Screenshot

image

Comment on lines +6 to +7
"firebase-admin": "^12.6.0",
"firebase-functions": "^6.0.1"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does this impact web frameworks too? What happens if we keep it as before?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@leoortizz Upgraded the extensions as well since last time an upgrade like this was done, it was requested the extensions to be done as well, see:
#7084 (review)

Can be removed if you think it's better!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not related to web frameworks, this is for ext:dev:init. We should keep these changes to templates/extensions

@emwp emwp enabled auto-merge (squash) October 11, 2024 22:20
@emwp emwp merged commit 372d0b1 into master Oct 16, 2024
40 of 41 checks passed
@joehan joehan deleted the chore/update-functions-template-dependencies branch October 16, 2024 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

package.json indicates an outdated version of firebase-functions.
3 participants