feat: add BigInt column type support and update related classes#167
feat: add BigInt column type support and update related classes#167ArnabChatterjee20k wants to merge 3 commits intomainfrom
Conversation
Greptile SummaryThis PR adds BigInt column type support across the migration library — introducing a new However, the PR has one blocking concern:
Additional non-blocking finding:
Confidence Score: 4/5Not safe to merge until The P1 issue in
Important Files Changed
Reviews (1): Last reviewed commit: "feat: add BigInt column type support and..." | Re-trigger Greptile |
| "ext-openssl": "*", | ||
| "appwrite/appwrite": "19.*", | ||
| "utopia-php/database": "5.*", | ||
| "utopia-php/database": "dev-big-init as 5.4", |
There was a problem hiding this comment.
Dev branch dependency should not be merged to main
The dependency is pinned to dev-big-init as 5.4, a development branch of utopia-php/database that has not been released as a stable version. This creates several production risks:
- The
big-initbranch can be force-pushed, rebased, or deleted at any time, breakingcomposer installfor anyone pulling this library. - The
"minimum-stability": "stable"declared incomposer.jsonis effectively bypassed via thestability-flagsoverride incomposer.lock(flag value20= dev-level stability). - Downstream consumers of this library will also pull the unstable dev branch.
This PR should be blocked until utopia-php/database has an official tagged release (e.g. 5.4.0) that includes BigInt support, at which point the constraint should be updated to "utopia-php/database": "5.4.*" or "5.*".
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
No description provided.