Skip to content

Task: 349 npm to pnpm - #352

Merged
acneville merged 12 commits into
mainfrom
task/349-npm-to-pnpm
Sep 28, 2026
Merged

acneville merged 12 commits into
mainfrom
task/349-npm-to-pnpm

Conversation

@acneville

Copy link
Copy Markdown
Contributor

Convert npm to pnpm and update READMEs accordingly.

@acneville
acneville marked this pull request as draft September 28, 2026 20:03
@acneville
acneville marked this pull request as ready for review September 28, 2026 20:03
@acneville acneville closed this Sep 28, 2026
@acneville acneville reopened this Sep 28, 2026
@acneville

Copy link
Copy Markdown
Contributor Author

@copilot review

@acneville

acneville commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor Author

Copilot is borked. Requesting dev team review. 🪦

@acneville acneville mentioned this pull request Sep 28, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@acneville

Copy link
Copy Markdown
Contributor Author

@copilot review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Two moderate issues remain unresolved, with an additional Storybook typing nit.

Review effort: Lite
Findings: 1 Medium severity · 1 Low severity

Open (2)

Comment thread package.json Outdated
Comment thread src/components/DownloadProjectData.stories.tsx
acneville and others added 2 commits September 28, 2026 15:58
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@acneville

Copy link
Copy Markdown
Contributor Author

found some more instances of npm when it should not be used so hold on momentarily...

@acneville

Copy link
Copy Markdown
Contributor Author

It's finally ready -- I think 😓

stdavis
stdavis previously approved these changes Sep 28, 2026
Comment thread README.md Outdated
@acneville
acneville merged commit 506330d into main Sep 28, 2026
7 checks passed
@acneville
acneville deleted the task/349-npm-to-pnpm branch September 28, 2026 23:07
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.

3 participants