Skip to content

feat: add support for pnpm #182

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

Merged
merged 6 commits into from
Apr 24, 2025
Merged

Conversation

ruromero
Copy link
Collaborator

Description

Fix #178

Checklist

  • I have followed this repository's contributing guidelines.
  • I will adhere to the project's code of conduct.

@ruromero ruromero requested a review from Strum355 April 23, 2025 08:27
@ruromero ruromero force-pushed the feature/pnpm branch 5 times, most recently from 36305a7 to c2c2cfa Compare April 23, 2025 14:27
@ruromero ruromero force-pushed the feature/pnpm branch 3 times, most recently from c2d4745 to 9861d64 Compare April 23, 2025 16:57
ruromero and others added 4 commits April 23, 2025 22:30
@ruromero ruromero enabled auto-merge (squash) April 23, 2025 21:13
Strum355
Strum355 previously approved these changes Apr 24, 2025
@Strum355
Copy link
Member

Not sure if I missed it somewhere, but I dont think I saw where the changes support custom paths for npm/pnpm via EXHORT_{,P}NPM_PATH. Was it removed and forgotten to be added into the rewritten version?

@ruromero
Copy link
Collaborator Author

Not sure if I missed it somewhere, but I dont think I saw where the changes support custom paths for npm/pnpm via EXHORT_{,P}NPM_PATH. Was it removed and forgotten to be added into the rewritten version?

You're right. I have added it.

Signed-off-by: Ruben Romero Montes <[email protected]>
Strum355
Strum355 previously approved these changes Apr 24, 2025
Signed-off-by: Ruben Romero Montes <[email protected]>
@ruromero ruromero merged commit fdeffcb into trustification:main Apr 24, 2025
4 checks passed
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.

Support for pnpm
2 participants