Skip to content

chore: retire config after its move to Appwrite - #373

Merged
ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-config
Oct 7, 2026
Merged

ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-config

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

Config now lives in Appwrite following appwrite/appwrite#14227. This removes packages/config so this repository stops discovering and publishing it during Split runs, regenerates the dependency graph, lists config with the packages maintained in Appwrite, and drops its CODEOWNERS line.

No package here requires utopia-php/config, so no constraints change.

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

Appwrite merged #14227 as a merge commit. Its post-merge Split pushed config, and the mirror's tree 30cdb54 matches Appwrite's packages/config exactly. config/2.1.0, which restores the static registry on Config, is cut from there. After this merges, confirm this repository's Split no longer includes config.

Only replication and reputation are left here.

Part of appwrite/appwrite#13828.

@hansi-codes

hansi-codes Bot commented Oct 7, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

No remaining config dependencies or references were found, and Split discovers packages from the remaining directories.

Removes the config package from this monorepo after its move to Appwrite, including its source, tests, and package configuration. Updates the README’s maintenance list and dependency graph, and removes config’s CODEOWNERS entry.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 3
File Change
packages/config/** Deletes the retired package’s source, tests, documentation, and configuration.
.github/CODEOWNERS Removes ownership for the deleted config directory.
README.md Lists config as maintained in Appwrite and removes its local dependency-graph node.

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

@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

@walter-o-brien

walter-o-brien Bot commented Oct 7, 2026

Copy link
Copy Markdown

✅ OpenCodeReview: Review complete: 0 finding(s) across 1 selected item(s).

@walter-o-brien walter-o-brien 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.

✅ OpenCodeReview found no issues in dd29335 and has no open findings on this pull request.

This is an automated review and does not replace a review by a maintainer.

@ChiragAgg5k
ChiragAgg5k merged commit 117efc7 into main Oct 7, 2026
8 checks passed
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