feat: accept plural orgs/users as canonical --for values#274
Conversation
Make the plural forms the canonical values for `--for` on `clerk enable billing` and `clerk disable billing`. The singular `org`/`user` continue to work as backward-compatible aliases. - Updated help text, examples, error messages, and tab-completion hints to use the plural forms. - Refreshed billing/orgs READMEs and cross-references in `clerk disable orgs` warnings. - Added a test asserting singular aliases still resolve correctly.
🦋 Changeset detectedLatest commit: 39352f2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughWalkthroughThis PR updates the Clerk CLI billing command to use plural forms ( Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Comment |
Co-authored-by: Rafael Thayto <rafa.thayto@gmail.com>
Summary
--forvalues onclerk enable billingandclerk disable billingfrom singular (org/user) to plural (orgs/users), since the plural forms read more naturally.clerk disable orgswarnings, plus tests that cover both canonical and alias inputs.Test plan
bun run test packages/cli-core/src/commands/billing/index.test.tsbun run typecheckbun run lintclerk enable billing --for orgsand--for orgboth still work