Skip to content

Update utopia-php/storage to 2.* for compatibility#174

Merged
TorstenDittmann merged 3 commits intomainfrom
update/storage-to-v2
Apr 27, 2026
Merged

Update utopia-php/storage to 2.* for compatibility#174
TorstenDittmann merged 3 commits intomainfrom
update/storage-to-v2

Conversation

@TorstenDittmann
Copy link
Copy Markdown
Contributor

@TorstenDittmann TorstenDittmann commented Apr 27, 2026

Summary

Updates the utopia-php/storage dependency constraint from 0.0.* to 2.* to maintain compatibility with the broader dependency tree.

Changes

  • Updated composer.json: utopia-php/storage 0.0.*2.*
  • Updated composer.lock to reflect the resolved dependency tree

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps Bot commented Apr 27, 2026

Greptile Summary

This PR updates the utopia-php/storage dependency from 0.* to 2.* and includes minor PHP formatting cleanups (blank-line removal, match( spacing, and // no break annotations). The lock file reflects several transitive changes: utopia-php/telemetry is downgraded from 0.3.0 to 0.2.0, laravel/pint (dev) from v1.29.1 to v1.2.1, and utopia-php/di, utopia-php/framework, and utopia-php/servers are dropped entirely from the resolved tree.

Confidence Score: 4/5

Safe to merge assuming the storage v2 API surface used by this codebase has been validated against the new major version.

Only P2 findings remain after previous review threads addressed the API-compatibility and telemetry-downgrade concerns. The single new finding (misleading // no break in match expressions) is purely cosmetic.

composer.lock — confirm utopia-php/telemetry 0.2.0 is compatible with all consumers in the resolved tree, and that ext-curl and ext-simplexml are present in all target environments.

Important Files Changed

Filename Overview
composer.json Updates utopia-php/storage constraint from 0.* to 2.*; the major-version jump is the core of this PR.
composer.lock Resolves storage 2.0.0; drops utopia-php/di, utopia-php/framework, utopia-php/servers; downgrades utopia-php/telemetry 0.3.0→0.2.0 and laravel/pint v1.29.1→v1.2.1 (dev); adds ext-curl and ext-simplexml as new transitive requirements.
src/Migration/Destinations/Appwrite.php Minor formatting cleanups (blank-line removal) and misleading // no break comments added to three match expressions.
src/Migration/Destinations/JSON.php Trailing blank line removed before closing brace; no logic changes.
src/Migration/Sources/Appwrite.php Minor formatting: blank lines removed, match( spacing fixed to match (; no logic changes.
src/Migration/Sources/Appwrite/Reader.php Trailing blank line removed from interface; no logic changes.
src/Migration/Resources/Sites/EnvVar.php Trailing blank line removed; no logic changes.
src/Migration/Sources/Firebase.php Trailing blank line removed; no logic changes.

Reviews (3): Last reviewed commit: "Merge branch 'main' into update/storage-..." | Re-trigger Greptile

Comment thread composer.json
Comment thread composer.lock
@TorstenDittmann TorstenDittmann merged commit 969dc94 into main Apr 27, 2026
4 checks passed
@TorstenDittmann TorstenDittmann deleted the update/storage-to-v2 branch April 27, 2026 12:42
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