Skip to content

chore: remove reserved-names module - #142

Merged
pyramation merged 1 commit into
mainfrom
devin/1792851950-remove-reserved-names
Oct 24, 2026
Merged

pyramation merged 1 commit into
mainfrom
devin/1792851950-remove-reserved-names

Conversation

@pyramation

Copy link
Copy Markdown
Contributor

Summary

Removes the reserved-names package, its CI matrix entry, and its README/MODULES.md entries.

Link to Devin session: https://app.devin.ai/sessions/a16778abc78648f586fa164fe0a06154
Open in Devin Desktop: https://app.devin.ai/desktop/session/a16778abc78648f586fa164fe0a06154?variant=devin
Requested by: @pyramation

@devin-ai-integration

Copy link
Copy Markdown
Contributor

I'll fix CI failures and address comments from users with write access. I'll skip comments containing "(aside)".

  • Disable automatic comment, CI, and merge conflict monitoring

@tenki-reviewer

tenki-reviewer Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Review complete. No issues found — approved ✅.


The PR deletes the entire packages/reserved-names module (deploy, verify, revert, tests, package metadata, and its generated bundle) from the pgpm workspace. No remaining references to reserved-names or its reserved_names schema exist in workspace manifests, CI workflows, root docs, or the lockfile, so the removal is self-consistent and safe to merge.

Files Change
packages/reserved-names/**, MODULES.md, README.md, .github/workflows/ci.yml Delete the reserved-names pgpm module and its documentation/CI references; no dangling references remain.

Reviewed commit: d599964

@pyramation
pyramation merged commit 90cd1a5 into main Oct 24, 2026
27 checks passed
@pyramation
pyramation deleted the devin/1792851950-remove-reserved-names branch October 24, 2026 20:01
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