Skip to content

chore: retire audit after its move to Appwrite - #369

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

ChiragAgg5k merged 1 commit into
mainfrom
chore/retire-audit

Conversation

@ChiragAgg5k

Copy link
Copy Markdown
Member

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

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

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

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

The open PR against the old copy (#206) is closed with a pointer to packages/audit.

Part of appwrite/appwrite#13828.

@hansi-codes

hansi-codes Bot commented Sep 28, 2026

Copy link
Copy Markdown

🟢 Tier S · Ready to merge

The remaining packages have no audit references, and Split discovers packages from the directories that remain.

Removes the audit package from this monorepo now that it is maintained in Appwrite. Updates the README’s Appwrite-maintained package list and regenerates the local dependency graph without audit.

Verdict New comments Fixed Still open
✅ Approved 0 0 0
📂 Walkthrough · 2
File Change
packages/audit/** Deletes the audit package, including its source, tests, manifest, and mirror workflow.
README.md Links to audit in Appwrite and removes it from the local dependency graph.

Reviewed c34c750 · 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 be97d87 into main Sep 28, 2026
7 of 8 checks passed
@ChiragAgg5k
ChiragAgg5k deleted the chore/retire-audit branch September 28, 2026 20:08
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