-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
chore(repo): switch package manager references from pnpm to utoo #5961
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Changes from all commits
Commits
Show all changes
14 commits
Select commit
Hold shift + click to select a range
ab76d57
chore(repo): switch package manager references from pnpm to utoo
killagu 26b1a40
fix(ci,docs): drop unsupported PUBLINT_PACK value, sync utoo/node ver…
killagu 8a96e3e
chore(repo): address CodeRabbit review feedback
killagu a0c2ddf
fix(ci): replace unsupported `utoo -r pack` with per-package npm pack
killagu 585d981
fix(docs,scripts): use \`ut\` CLI alias and real script names
killagu 80b413b
fix(ci): restore pnpm for E2E pack step
killagu f88b87b
fix(ci): pin pnpm version in E2E setup-pnpm action
killagu 2a4c085
docs: fix script descriptions and prepend corepack bootstrap
killagu 38ee2ad
fix(ci): disable pnpm's packageManager strict check for the pack step
killagu e38249c
docs: fix \`ut test\` → \`ut run test\` and Dumi → VitePress reference
killagu 5f8b4c3
fix(ci): move pnpm setup after \`ut install\` to avoid SIGPIPE
killagu 842a307
ci: retry \`ut install\` up to 3 times for transient SIGPIPE (141)
killagu c522197
fix(ci): use correct env name for pnpm packageManager strict bypass
killagu 5c856b3
fix(ci): run \`pnpm install\` before \`pnpm -r pack\` so workspace de…
killagu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -62,8 +62,8 @@ jobs: | |
| fetch-depth: 0 | ||
| token: ${{ secrets.GIT_TOKEN }} | ||
|
|
||
| - name: Setup pnpm | ||
| uses: pnpm/action-setup@41ff72655975bd51cab0327fa583b6e92b6d3061 # v4 | ||
| - name: Setup utoo | ||
| uses: utooland/setup-utoo@3a51006d0b66afcc32d1b9177a4b200b74f4a8cb # main | ||
|
|
||
| - name: Setup Node.js | ||
| uses: actions/setup-node@6044e13b5dc448c55e2357c09f80417699197238 # v6 | ||
|
|
@@ -72,7 +72,7 @@ jobs: | |
| registry-url: 'https://registry.npmjs.org' | ||
|
|
||
| - name: Install dependencies | ||
| run: pnpm install --no-frozen-lockfile | ||
| run: ut install --from pnpm | ||
|
Comment on lines
74
to
+75
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Add retry parity for Line 75 is a single-attempt install, while the same command was made retry-safe in other workflows for transient failures. This can cause avoidable manual release failures. Suggested fix- - name: Install dependencies
- run: ut install --from pnpm
+ - name: Install dependencies
+ # Retry to absorb transient `exit 141` (SIGPIPE) from `ut install`.
+ run: ut install --from pnpm || (sleep 5 && ut install --from pnpm) || (sleep 10 && ut install --from pnpm)🤖 Prompt for AI Agents |
||
|
|
||
| - name: Configure Git | ||
| run: | | ||
|
|
@@ -149,7 +149,7 @@ jobs: | |
| git push origin ${{ github.event.inputs.branch }} --tags | ||
|
|
||
| - name: Run build | ||
| run: pnpm build | ||
| run: ut run build | ||
|
|
||
| - name: Publish packages (dry run) | ||
| if: ${{ github.event.inputs.dry_run == 'true' }} | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
可以去掉了,让错误直接爆出来吧