fix(deps): update dependency google/cloud-firestore to v2#2196
Conversation
There was a problem hiding this comment.
Code Review
This pull request updates the google/cloud-firestore dependency from version 1.25 to 2.0 in composer.json. The review feedback suggests moving this dependency to the require-dev section to optimize the Cloud Function's deployment size and cold start performance, as it is only utilized in deployment tests rather than the production code.
| "require": { | ||
| "google/cloud-functions-framework": "^1.0.0", | ||
| "google/cloud-firestore": "^1.25" | ||
| "google/cloud-firestore": "^2.0" |
There was a problem hiding this comment.
The google/cloud-firestore dependency is only used in the deployment tests (DeployTest.php) and not in the actual function code (index.php). To optimize the Cloud Function's deployment size and cold start performance, this dependency should be moved from the require section to require-dev. Additionally, note that version 2.0 of this library requires PHP 8.0 or higher. If the function is intended to run on older PHP runtimes, this upgrade might cause compatibility issues unless the dependency is moved to require-dev (ensuring it is not installed in the production environment).
This PR contains the following updates:
^1.25→^2.0Release Notes
googleapis/google-cloud-php-firestore (google/cloud-firestore)
v2.0.0: google/cloud-firestore v2.0.0Compare Source
Miscellaneous Chores
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR was generated by Mend Renovate. View the repository job log.