[Automated] Update pulumi CLI Options - #4736
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (8)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (1)
Included review availability: Your plan provides up to 4 included reviews per hour; 1 remains after this review. 📝 WalkthroughWalkthroughThe Pulumi public API baselines update enum values and remove obsolete enum members across Insights, organization, usage, and stack-schedule APIs. ChangesPulumi API baselines
Estimated code review effort: 2 (Simple) | ~10 minutes Merge Risk: ⚪ Minimal · up to This updates generated Pulumi CLI API baselines for the newer CLI output. The documented enum ordinal and API-surface breaking changes are intentional, with no remaining merge-readiness risk identified. Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: cdc5c710ed
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Review: PR #4736 — Pulumi enum options regenerationThis PR is a fully auto-generated update to the Pulumi CLI options package, touching only:
No hand-written code, generator source, or scraper logic was touched, so the "never hand-tweak generated options" and "fix in the scraper/generator" rules from CLAUDE.md don't come into play here — this is the generator's own output. Verification performed:
No prior substantive review comments exist on this PR to reconcile against (only a rate-limited CodeRabbit notice). Verdict: No correctness bugs, no reuse/simplification/efficiency issues, and no CLAUDE.md violations. This is a clean, mechanical, low-risk regeneration. |
|
Latest-options audit: this is the intentional one-time enum reorder from #4661 / merged #4663. That decision explicitly accepts the v4 ordinal break and prohibits restoring prior-output preservation. Named members still map through their unchanged EnumValue attributes; numeric/default consumers must account for the documented breaking change. No handwritten ordinal assignments or compatibility shims will be added. Resolving this review concern under the accepted generator policy. |
cdc5c71 to
f979e61
Compare
Code reviewThis is an automated ModularPipelines.OptionsGenerator update to the Pulumi enum options, picking up upstream CLI help changes for v3.261.0. I reviewed the full diff. What changed:
Assessment:
No issues found. Checked for correctness of the generated changes and CLAUDE.md compliance. |
f979e61 to
ecafef7
Compare
|
Reviewed this automated Pulumi options regeneration PR. The entire diff is mechanical: enum members are being reordered from CLI-declaration order to alphabetical order (by their Everything checked out:
No actionable findings. |
Summary
This PR contains automatically generated updates to pulumi CLI options classes.
The generator scraped the latest CLI help output from the installed tool.
Changes
Assembly-wide public API impact
Affected API families:
Pulumi.Breaking changes are present. Consumers may need to update method arguments, option property types or nullability, enum members, and references to removed APIs.
Representative removed or changed members:
ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.AzureNative = 2 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Gcp = 1 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Kubernetes = 4 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Oci = 3 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewScanSchedule.Daily = 2 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewScanScheduleRepresentative added members:
ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.AzureNative = 1 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Gcp = 2 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Kubernetes = 3 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProvider.Oci = 4 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewProviderModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewScanSchedule.Daily = 1 -> ModularPipelines.Pulumi.Enums.PulumiInsightsAccountNewScanScheduleCommand coverage
Command coverage report:
Verification
🤖 Generated with ModularPipelines.OptionsGenerator
Summary by CodeRabbit
New Features
Breaking Changes