Skip to content
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

enforce pnpm a bit more #82

Merged
merged 3 commits into from
Jul 31, 2023
Merged

enforce pnpm a bit more #82

merged 3 commits into from
Jul 31, 2023

Conversation

andersonjoseph
Copy link
Contributor

closes: #81


btw: pnpm has a tool to force a specific package manager called pnpm/only-allow if you want to include it in the project as well, I can include it in this PR

Copy link
Owner

@kytta kytta left a comment

Choose a reason for hiding this comment

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

Great job! See one comment about the wording in CONTRIBUTING.md.

And speaking of only-allow, I'd love to see it here, too!

CONTRIBUTING.md Outdated Show resolved Hide resolved
package.json Outdated Show resolved Hide resolved
@kytta kytta merged commit b233187 into kytta:main Jul 31, 2023
2 checks passed
@kytta
Copy link
Owner

kytta commented Jul 31, 2023

Thanks a lot!

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.

Enforce PNPM a bit more
2 participants