Skip to content

Conversation

@rxmarbles
Copy link
Member

@rxmarbles rxmarbles commented Aug 29, 2025

Summary

Migrate to pnpm to manage packages

Changeset

  • [ ] Changeset added (docs)

Test Plan

All CI tests pass

@rxmarbles rxmarbles requested a review from a team as a code owner August 29, 2025 05:42
@github-actions
Copy link
Contributor

'This PR is stale as it has been open for 30 days with no activity. Remove stale label or comment or this will be closed in 5 days.'

@github-actions github-actions bot added the stale label Sep 29, 2025
Copy link
Contributor

@agerard-godaddy agerard-godaddy left a comment

Choose a reason for hiding this comment

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

pnpm changes look good, but you have some Gasket stuff copied over in the new documents,

Copy link

@mmicco-godaddy mmicco-godaddy left a comment

Choose a reason for hiding this comment

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

Just comments.

@github-actions github-actions bot removed the stale label Sep 30, 2025
@rxmarbles rxmarbles merged commit ef5340b into main Sep 30, 2025
3 checks passed
@rxmarbles rxmarbles deleted the rm/pnpm branch September 30, 2025 18:15
wcole1-godaddy pushed a commit that referenced this pull request Oct 13, 2025
* chore: add pnpm workspace configuration to manage packages

* fix(ci): update test command to use pnpm instead of npm

* chore: update governance and contributing guidelines, remove package-lock.json
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.

6 participants