Skip to content

feat: add preinstall script to enforce pnpm usage as the package manager. - #81

Merged
SujithThirumalaisamy merged 5 commits into
code100x:mainfrom
geekyharsh05:feat/preinstall-script
Sep 11, 2024
Merged

feat: add preinstall script to enforce pnpm usage as the package manager.#81
SujithThirumalaisamy merged 5 commits into
code100x:mainfrom
geekyharsh05:feat/preinstall-script

Conversation

@geekyharsh05

Copy link
Copy Markdown
Contributor

This pr fixes the issue #34:

  • Only allow pnpm as the package manager in the project

Screenshots:

Yarn:

Screenshot 2024-09-11 at 2 05 21 AM

NPM:

Screenshot 2024-09-11 at 2 05 39 AM

@geekyharsh05

Copy link
Copy Markdown
Contributor Author

@SujithThirumalaisamy Can you please review this.

@nawinsharma

Copy link
Copy Markdown

@geekyharsh05 LGTM !

@SujithThirumalaisamy

Copy link
Copy Markdown
Contributor

Can you please fix the conflicts

@SujithThirumalaisamy
SujithThirumalaisamy merged commit f6f7539 into code100x:main Sep 11, 2024
@geekyharsh05

Copy link
Copy Markdown
Contributor Author

@SujithThirumalaisamy Done!

@SujithThirumalaisamy

Copy link
Copy Markdown
Contributor

Thankyou for the quick resolve. Sorry was fixing something 👍

@geekyharsh05

Copy link
Copy Markdown
Contributor Author

@SujithThirumalaisamy Yeah, Thank You.

@geekyharsh05
geekyharsh05 deleted the feat/preinstall-script branch September 13, 2024 17:51
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