Skip to content

chore: retire vcs after its move to Appwrite - #367

Merged
ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-vcs
Sep 28, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-vcs

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

VCS now lives in Appwrite following appwrite/appwrite#13932. Remove packages/vcs so this repository stops discovering and publishing it during Split runs. Refresh the local dependency graph and list vcs with the packages maintained in Appwrite.

No package here requires utopia-php/vcs, so no constraint changes.

Validation: bin/monorepo validate and git diff --check pass.

Appwrite merged #13932 as a merge commit, and its post-merge Split pushed vcs: the mirror's tree 847fb5e exactly matches Appwrite's packages/vcs. After merge, confirm this repository's Split no longer includes vcs.

Open PRs against the old copy (#264, #220) are closed with a pointer to packages/vcs.

Part of appwrite/appwrite#13828.

@hansi-codes

hansi-codes Bot commented Sep 28, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

Package discovery uses the remaining package directories, and no local Composer manifest depends on VCS.

Retires the local VCS package after its move to Appwrite. The README now links to the Appwrite copy and removes VCS from the local dependency graph.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 2
File Change
README.md Adds VCS to the Appwrite-maintained package list and removes its local graph node.
packages/vcs/ Removes the local VCS package so repository tooling no longer discovers it for splitting.

Reviewed 65eb49a · Details · Comment @hansi-codes review to re-run, or mention @hansi-codes with a question.

@walter-o-brien

Copy link
Copy Markdown

✅ OpenCodeReview: Review skipped: no items were selected.

@hansi-codes hansi-codes Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Tier S · Looks good to merge. Summary

@ChiragAgg5k
ChiragAgg5k merged commit 8f1fa98 into main Sep 28, 2026
7 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the chore/retire-vcs branch September 28, 2026 15:19
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.

1 participant