Skip to content

chore: retire queue after its move to Appwrite - #366

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

ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-queue

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

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

platform here still requires utopia-php/queue; it now resolves it from Packagist, which the mirror keeps serving, so no constraint changes.

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

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

Open PRs against the old copy (#348, #339, #325, #146) are closed with a pointer to packages/queue.

Part of appwrite/appwrite#13828.

@hansi-codes

hansi-codes Bot commented Sep 28, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The removed package is no longer discovered from the packages directory, and the remaining platform dependency is still declared for registry resolution.

Removes the queue package from this monorepo and updates the README to list queue among the libraries maintained in Appwrite. The local dependency graph now reflects that platform's queue dependency is resolved externally rather than from a sibling package.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 2
File Change
packages/queue Removes the queue package and its mirror workflow from this repository.
README.md Adds queue to the Appwrite-maintained package list and refreshes the local dependency graph.

Reviewed 5d3a588 · 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

Copy link
Copy Markdown

✅ OpenCodeReview: Review skipped: no items were selected.

@ChiragAgg5k
ChiragAgg5k merged commit a9b5f4f into main Sep 28, 2026
7 of 9 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the chore/retire-queue branch September 28, 2026 14:09
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