Update telemetry dependency constraint#83
Conversation
Greptile SummaryThis PR relaxes the
Confidence Score: 4/5Safe to merge; the actual installed dependency set is unchanged and no runtime behavior is affected. The only change is a constraint notation swap that resolves to the same version range, so there is no functional impact. The stated goal of relaxing the constraint to ease cross-library alignment is not achieved by this change, since No files require special attention; both changed files are purely declarative dependency metadata. Important Files Changed
Reviews (1): Last reviewed commit: "Update telemetry dependency constraint" | Re-trigger Greptile |
| "utopia-php/servers": "0.4.*", | ||
| "utopia-php/pools": "1.*", | ||
| "utopia-php/telemetry": "0.4.*", | ||
| "utopia-php/telemetry": "^0.4", |
There was a problem hiding this comment.
Constraint change is semantically a no-op in Composer
In Composer, ^0.4 for a pre-1.0 package resolves to >=0.4.0 <0.5.0 — exactly the same range as 0.4.*. The PR description frames this as "relaxing" the constraint, but the set of installable versions is unchanged. If the intent is to also allow 0.5.x or higher minor releases in the future without another constraint bump, a constraint like >=0.4 would be needed instead.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
Summary
utopia-php/telemetryfrom0.4.*to^0.4.composer.lockcontent hash for the root constraint change.Why
Circuit breaker
0.3.1uses the telemetry0.4lazy metric API. Keeping Utopia libraries on narrow telemetry constraints makes dependency alignment harder across the Appwrite stack.Tests
composer validate --strictcomposer lintcomposer test(fails in local environment becauseamqp:5672is not resolvable)