Skip to content

chore(ci): support ecosystem-ci use different npm in different repo #10962

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

Merged
merged 17 commits into from
Jul 11, 2025

Conversation

stormslowly
Copy link
Contributor

@stormslowly stormslowly commented Jul 9, 2025

Summary

Adjust workflow to support ecosytem CI testing with different version of pnpm.

  1. Use verdaccio to setup a mock registery foring publish the current rspack's npm packages.
  2. Run ecosystem-ci under the mock registery, as all the current rspack's packages are published. So no matter what the underhood package manager is, it will work all fine.

Related links

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Copy link

netlify bot commented Jul 9, 2025

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit 0dc74a4
🔍 Latest deploy log https://app.netlify.com/projects/rspack/deploys/6870b41110db24000847c719
😎 Deploy Preview https://deploy-preview-10962--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@github-actions github-actions bot added release: refactor team The issue/pr is created by the member of Rspack. labels Jul 9, 2025
@stormslowly stormslowly changed the title refactor: use testing branch chore(ci): remove ecosystem-ci pnpm overrides Jul 9, 2025
Copy link
Contributor

github-actions bot commented Jul 9, 2025

📦 Binary Size-limit

Comparing 0dc74a4 to chore: add default-feaults=false check (#10984) by hardfist

🙈 Size remains the same at 57.48MB

Copy link

codspeed-hq bot commented Jul 9, 2025

CodSpeed Performance Report

Merging #10962 will not alter performance

Comparing fix/ecosystem_ci_pnpm_overrides (0dc74a4) with main (5604372)

Summary

✅ 16 untouched benchmarks

@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 9, 2025
Copy link
Contributor

github-actions bot commented Jul 9, 2025

📝 Ecosystem CI detail: Open

suite result
rspress ✅ success
rsdoctor ❌ failure
rslib ✅ success
modernjs ❌ failure
examples ✅ success
devserver ✅ success
nuxt ✅ success
rsbuild ✅ success
plugin ✅ success
lynx-stack ✅ success

@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 9, 2025
hardfist
hardfist previously approved these changes Jul 9, 2025
@hardfist hardfist marked this pull request as ready for review July 9, 2025 11:00
@stormslowly stormslowly marked this pull request as draft July 9, 2025 11:05
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@stormslowly stormslowly force-pushed the fix/ecosystem_ci_pnpm_overrides branch from d4abd75 to c4de2be Compare July 10, 2025 08:26
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@stormslowly stormslowly force-pushed the fix/ecosystem_ci_pnpm_overrides branch from c4de2be to c9e05b7 Compare July 10, 2025 08:33
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
Copy link
Contributor

github-actions bot commented Jul 10, 2025

📝 Ecosystem CI detail: Open

suite result
rsdoctor ✅ success
rsbuild ✅ success
rslib ✅ success
modernjs ✅ success
nuxt ✅ success
rspress ✅ success
examples ✅ success
plugin ✅ success
lynx-stack ✅ success
devserver ✅ success

@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 10, 2025
Copy link
Contributor

github-actions bot commented Jul 10, 2025

📝 Ecosystem CI detail: Open

suite result
rspress ✅ success
examples ✅ success
modernjs ✅ success
rsbuild ✅ success
rsdoctor ✅ success
rslib ✅ success
devserver ✅ success
plugin ✅ success
lynx-stack ✅ success
nuxt ✅ success

@web-infra-dev web-infra-dev deleted a comment from github-actions bot Jul 11, 2025
@stormslowly stormslowly marked this pull request as ready for review July 11, 2025 07:03
Copy link
Contributor

github-actions bot commented Jul 11, 2025

📝 Ecosystem CI detail: Open

suite result
rslib ✅ success
rspress ✅ success
rsdoctor ✅ success
rsbuild ✅ success
nuxt ✅ success
lynx-stack ✅ success
devserver ✅ success
modernjs ❌ failure
examples ✅ success
plugin ✅ success

@stormslowly stormslowly changed the title chore(ci): remove ecosystem-ci pnpm overrides chore(ci): support ecosystem-ci use different npm in different repo Jul 11, 2025
@stormslowly stormslowly merged commit 5bf97de into main Jul 11, 2025
86 checks passed
@stormslowly stormslowly deleted the fix/ecosystem_ci_pnpm_overrides branch July 11, 2025 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team The issue/pr is created by the member of Rspack.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants