chore: remove pre-release CI for extension, too much work and doesn't make sense for us#1129
Conversation
… make sense for us
|
|
Caution Review failedThe pull request is closed. WalkthroughRemoves the RC-specific script and the extension pre-release workflow, and updates the main pre-release workflow to run “npx changeset publish” directly, dropping marketplace publishing secrets from that step. Changes
Sequence Diagram(s)sequenceDiagram
participant GitHub Actions as pre-release.yml
participant Changesets as npx changeset publish
participant npm Registry as npm
GitHub Actions->>Changesets: Run publish
Changesets->>npm Registry: Publish packages
npm Registry-->>Changesets: Publish result
Changesets-->>GitHub Actions: Exit code/logs
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (3)
✨ Finishing Touches
🧪 Generate unit tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
Status, Documentation and Community
|
… make sense for us (eyaltoledano#1129)
… make sense for us (eyaltoledano#1129)
… make sense for us (eyaltoledano#1129)
… make sense for us (eyaltoledano#1129)
What type of PR is this?
Description
Related Issues
How to Test This
# Example commands or stepsExpected result:
Contributor Checklist
npm run changesetnpm testnpm run format-check(ornpm run formatto fix)Changelog Entry
For Maintainers
Summary by CodeRabbit