chore(deps): update phpswoole/swoole docker tag to v6 - #1641
chore(deps): update phpswoole/swoole docker tag to v6#1641renovate-appwrite[bot] wants to merge 1 commit into
Conversation
Greptile SummaryThis automated Renovate PR bumps the
Confidence Score: 4/5The Dockerfile change itself is minimal, but this is a major Swoole version jump and the mock server's CI tests should be run and pass before merging to confirm the utopia-php/swoole wrapper and WebSocket adapter remain fully functional on Swoole 6. The mock server code does not directly use the Swoole APIs that were removed or restructured in v6, and the single changed line is straightforward. The main open question is whether utopia-php/swoole 0.8.* and utopia-php/websocket ^1.0 are tested and compatible with Swoole 6; without passing CI evidence this remains unverified. The swoole/ide-helper pin mismatch is a development tooling gap rather than a runtime issue. mock-server/composer.json (not in the diff) — the swoole/ide-helper dev dependency should be updated to a Swoole 6-compatible version. Important Files Changed
Prompt To Fix All With AIFix the following 1 code review issue. Work through them one at a time, proposing concise fixes.
---
### Issue 1 of 1
mock-server/Dockerfile:17
**`swoole/ide-helper` dev dependency is now mismatched**
`mock-server/composer.json` pins `"swoole/ide-helper": "5.1.8"` but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The `composer.json` `"require-dev"` entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.
Reviews (1): Last reviewed commit: "chore(deps): update phpswoole/swoole doc..." | Re-trigger Greptile |
| `if [ "$TESTING" != "true" ]; then echo "--no-dev"; fi` | ||
|
|
||
| FROM phpswoole/swoole:5.1.8-php8.3-alpine as final | ||
| FROM phpswoole/swoole:6.2.1-php8.3-alpine as final |
There was a problem hiding this comment.
swoole/ide-helper dev dependency is now mismatched
mock-server/composer.json pins "swoole/ide-helper": "5.1.8" but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The composer.json "require-dev" entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.
Prompt To Fix With AI
This is a comment left during a code review.
Path: mock-server/Dockerfile
Line: 17
Comment:
**`swoole/ide-helper` dev dependency is now mismatched**
`mock-server/composer.json` pins `"swoole/ide-helper": "5.1.8"` but the runtime image is now Swoole 6.2.1. The IDE helper provides stubs that mirror the actual Swoole extension API, so running it against the wrong major version means IDE type-checking and autocompletion will reflect the old 5.x surface (including APIs removed in 6.x). The `composer.json` `"require-dev"` entry should be updated to a Swoole 6-compatible IDE helper version alongside this image bump.
How can I resolve this? If you propose a fix, please make it concise.
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
Closing: CI is failing / known incompatible with current SDK templates and tooling. Superseded by Renovate config guards on cursor/renovate-dashboard-debug. |
Renovate Ignore NotificationBecause you closed this PR without merging, Renovate will ignore this update. You will not get PRs for any future If you accidentally closed this PR, or if you changed your mind: rename this PR to get a fresh replacement PR. |
This PR contains the following updates:
5.1.8-php8.3-alpine→6.2.1-php8.3-alpineConfiguration
📅 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 has been generated by Mend Renovate.