Skip to content

fix(deps): upgrade downgraded deps#2767

Merged
CorieW merged 3 commits intonextfrom
@invertase/fix-dependencies
Apr 15, 2026
Merged

fix(deps): upgrade downgraded deps#2767
CorieW merged 3 commits intonextfrom
@invertase/fix-dependencies

Conversation

@CorieW
Copy link
Copy Markdown
Member

@CorieW CorieW commented Apr 14, 2026

No description provided.

@CorieW CorieW requested a review from a team as a code owner April 14, 2026 15:01
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the firebase-functions-test dependency to version ^3.2.0 across several function directories. The reviewer noted that the package-lock.json files appear to have been manually edited rather than updated via npm install, which will cause npm ci to fail in CI/CD environments. Additionally, the major version upgrade of firebase-functions-test requires firebase-functions version 4.0.0 or higher to prevent test execution failures due to significant API changes.

Comment thread delete-user-data/functions/package-lock.json
Comment thread firestore-bigquery-export/functions/package-lock.json
Comment thread firestore-send-email/functions/package-lock.json
Comment thread firestore-translate-text/functions/package-lock.json
Comment thread storage-resize-images/functions/package-lock.json
Comment thread delete-user-data/functions/package.json
Comment thread firestore-bigquery-export/functions/package.json
Comment thread firestore-send-email/functions/package.json
Comment thread storage-resize-images/functions/package.json
Copy link
Copy Markdown
Contributor

@cabljac cabljac left a comment

Choose a reason for hiding this comment

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

lgtm!

@CorieW CorieW merged commit 58896ff into next Apr 15, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants