Greptile Summary

This PR adds a shouldSkipGitDeployment() helper that checks for [skip deploy] or [skip deployment] in the commit message (case-insensitive), and short-circuits createGitDeployments before any build is queued. The skip check is now correctly placed after $logBase is set to the real repository ID, so the span key issue from the previous review thread is addressed. However, the six lines immediately following the skip check (lines 79–87) are a verbatim copy of the block that already ran at lines 64–72; they should be removed.

Confidence Score: 5/5

Safe to merge after removing the duplicate variable/span block; the core skip logic and regex are correct.

The only finding is a P2 code-duplication issue (redundant variable assignments and span calls) that does not affect deployment behavior or data correctness. All prior concerns about the repo.unknown span key are resolved.

src/Appwrite/Platform/Modules/VCS/Http/GitHub/Deployment.php — lines 79–87 should be deleted.

Important Files Changed

_{Reviews (3): Last reviewed commit: "Merge branch '1.9.x' into feat-skip-depl..." | Re-trigger Greptile}

🔄 PHP-Retry Summary

Flaky tests detected across commits:

@greptile

✨ Benchmark results

• Requests per second: 1,691
• Requests with 200 status code: 304,348
• P99 latency: 0.109456549

⚡ Benchmark Comparison