Skip to content

Add .pnpm-store to Node.gitignore#4707

Merged
thorrsson merged 1 commit intogithub:mainfrom
stone-w4tch3r:patch-1
Feb 12, 2026
Merged

Add .pnpm-store to Node.gitignore#4707
thorrsson merged 1 commit intogithub:mainfrom
stone-w4tch3r:patch-1

Conversation

@stone-w4tch3r
Copy link
Contributor

Reasons for making this change

Due to current bugs in pnpm, it sometimes creates .pnpm-store directory at project root instead of global pnpm cache dir. Since it's chache, it is not needed in git repo, and it has enormous size, like that of node_modules/

Links to documentation supporting these rule changes

In this 51 start repo with pnpm workspace example we can see that .pnpm-store is ignored

Related issues:
pnpm/pnpm#7050 (comment)
gravitational/teleport#46019
gravitational/teleport#54701
pnpm/pnpm#7100

Merge and Approval Steps

  • Confirm that you've read the contribution guidelines and ensured your PR aligns
  • Ensure CI is passing
  • Get a review and Approval from one of the maintainers

@stone-w4tch3r stone-w4tch3r requested a review from a team as a code owner August 11, 2025 22:21
@yorickdewid yorickdewid mentioned this pull request Aug 19, 2025
3 tasks
@github-actions
Copy link

This PR is stale because there have been no updates in 90 days. It will close after 180 days of inactivity. Leave a comment if you want to keep it open 😄

@github-actions github-actions bot added the stale label Nov 11, 2025
@stone-w4tch3r
Copy link
Contributor Author

Is there a chance for it to be merged?

@github-actions github-actions bot removed the stale label Nov 14, 2025
@github-actions
Copy link

This PR is stale because there have been no updates in 90 days. It will close after 365 days of inactivity. Leave a comment if you want to keep it open.

@github-actions github-actions bot added the stale label Feb 12, 2026
@thorrsson thorrsson merged commit b4105e7 into github:main Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants