Skip to content

chore(deps): use setup-vp instead of actions/setup-node - #931

Merged
fengmk2 merged 3 commits into
masterfrom
renovate/actions-setup-node-digest
Oct 8, 2026
Merged

fengmk2 merged 3 commits into
masterfrom
renovate/actions-setup-node-digest

Conversation

@renovate

@renovate renovate Bot commented Oct 8, 2026 •

Copy link
Copy Markdown
Contributor

No description provided.

@coderabbitai

coderabbitai Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 471dcefa-1a1a-4980-9bc0-4c41ed357921

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pkg-pr-new

pkg-pr-new Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/node-modules/urllib@931

commit: d9c5a8d

@codecov

codecov Bot commented Oct 8, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 94.77%. Comparing base (da8aac1) to head (d9c5a8d).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #931   +/-   ##
=======================================
  Coverage   94.77%   94.77%           
=======================================
  Files          10       10           
  Lines         746      746           
  Branches      235      235           
=======================================
  Hits          707      707           
  Misses         36       36           
  Partials        3        3           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@fengmk2

fengmk2 commented Oct 8, 2026

Copy link
Copy Markdown
Member

@copilot 应该使用 setup-vp 代替?

Co-authored-by: fengmk2 <156269+fengmk2@users.noreply.github.com>

Copilot AI commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

@copilot 应该使用 setup-vp 代替?

Agreed, done in 710672e. codspeed.yml now uses voidzero-dev/setup-vp (same pin as nodejs.yml) instead of pnpm/action-setup, actions/setup-node and the manual install step. The benchmark step runs vp test bench --run. I didn't run the workflow, so it's untested.

Copilot AI requested a review from fengmk2 October 8, 2026 13:45
@renovate

renovate Bot commented Oct 8, 2026

Copy link
Copy Markdown
Contributor Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@fengmk2 fengmk2 changed the title chore(deps): update actions/setup-node digest to 949feb2 chore(deps): use setup-vp instead of actions/setup-node Oct 8, 2026
@fengmk2
fengmk2 merged commit 75010e6 into master Oct 8, 2026
17 checks passed
@fengmk2
fengmk2 deleted the renovate/actions-setup-node-digest branch October 8, 2026 13:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants