fix: Fix the path error in the release script and change mobile/dist in the pub:theme command to theme-mobile/dist - #29
Conversation
|
Caution Review failedThe pull request is closed. WalkthroughThe CI workflow’s publish step now calls new pnpm scripts (pub, pub:theme) instead of direct path-based publish commands. Root package.json adds/updates these scripts to use pnpm filters. Versions for packages/mobile and packages/theme-mobile were bumped from 1.0.0-alpha.7 to 1.0.0-alpha.9. Changes
Sequence Diagram(s)sequenceDiagram
participant Dev as Maintainer
participant GH as GitHub Actions
participant PN as pnpm scripts
participant NPM as npm Registry
Dev->>GH: Dispatch publish workflow (with tag)
GH->>PN: Run "pnpm pub --tag <tag>"
PN->>NPM: Publish @opentiny/vue-mobile
GH->>PN: Run "pnpm pub:theme --tag <tag>"
PN->>NPM: Publish theme-mobile/dist
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
Poem
Tip 🔌 Remote MCP (Model Context Protocol) integration is now available!Pro plan users can now connect to remote MCP servers from the Integrations page. Connect with popular remote MCPs such as Notion and Linear to add more context to your reviews and chats. 📜 Recent review detailsConfiguration used: CodeRabbit UI 💡 Knowledge Base configuration:
You can enable these sources in your CodeRabbit configuration. 📒 Files selected for processing (4)
✨ Finishing Touches🧪 Generate unit tests
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. 🪧 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
CodeRabbit Configuration File (
|
fix: 修复发布脚本中的路径错误,将pub:theme命令中的mobile/dist更改为theme-mobile/dist
PR
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: N/A
What is the new behavior?
Does this PR introduce a breaking change?
Other information
Summary by CodeRabbit