Skip to content

feat: add support for pnpm#1629

Open
neilime wants to merge 1 commit intomainfrom
feat/supports-pnpm
Open

feat: add support for pnpm#1629
neilime wants to merge 1 commit intomainfrom
feat/supports-pnpm

Conversation

@neilime
Copy link
Copy Markdown
Member

@neilime neilime commented Sep 29, 2025

No description provided.

@neilime neilime self-assigned this Sep 29, 2025
@neilime neilime force-pushed the feat/supports-pnpm branch 3 times, most recently from 42c1287 to 4263c23 Compare September 29, 2025 16:33
Copy link
Copy Markdown
Member Author

@neilime neilime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot finialize the work: fix build make tests passing

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 7, 2025

@neilime I've opened a new pull request, #1659, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
Copy Markdown
Member Author

@neilime neilime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@copilot migrate this repo to use pnpm instead of npm

Copy link
Copy Markdown
Contributor

Copilot AI commented Nov 7, 2025

@neilime I've opened a new pull request, #1660, to work on those changes. Once the pull request is ready, I'll request review from you.

@neilime neilime force-pushed the feat/supports-pnpm branch 5 times, most recently from 2054ecc to c55bc1a Compare November 15, 2025 06:45
@codecov
Copy link
Copy Markdown

codecov bot commented Nov 15, 2025

Codecov Report

❌ Patch coverage is 84.70588% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 87.97%. Comparing base (c5bb0c1) to head (67753df).

Files with missing lines Patch % Lines
...vices/package-manager/PnpmPackageManagerAdapter.ts 75.00% 11 Missing ⚠️
packages/core/src/tests/workspace-root.ts 86.66% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1629      +/-   ##
==========================================
- Coverage   88.41%   87.97%   -0.44%     
==========================================
  Files          37       36       -1     
  Lines        1027     1073      +46     
  Branches      193      202       +9     
==========================================
+ Hits          908      944      +36     
- Misses        118      128      +10     
  Partials        1        1              

☔ View full report in Codecov by Sentry.
📢 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.

@neilime neilime force-pushed the feat/supports-pnpm branch 2 times, most recently from 7c18d34 to e22c9cf Compare December 4, 2025 13:51
@escemi-tech escemi-tech deleted a comment from github-actions bot Feb 3, 2026
@neilime neilime force-pushed the feat/supports-pnpm branch 2 times, most recently from 569c095 to a573995 Compare February 10, 2026 08:27
@neilime neilime force-pushed the main branch 4 times, most recently from 8a30ce6 to c052b8c Compare February 10, 2026 11:21
@neilime neilime force-pushed the feat/supports-pnpm branch 2 times, most recently from 959aa64 to 63e3f62 Compare February 10, 2026 18:34
@neilime neilime force-pushed the feat/supports-pnpm branch 2 times, most recently from 5718e05 to 1f558de Compare February 17, 2026 14:33
Signed-off-by: Emilien Escalle <[email protected]>
@neilime neilime force-pushed the feat/supports-pnpm branch from 1f558de to 67753df Compare March 5, 2026 08:32
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