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

chore: add onlyBuiltDependencies list #4702

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

btea
Copy link
Contributor

@btea btea commented Feb 21, 2025

After pnpm 10 lifecycle scripts of dependencies are not executed during installation by default! https://github.com/pnpm/pnpm/releases/tag/v10.0.0

close #xxx

Pull Request Checklist

  • Related tickets or issues have been linked in the PR description (or missing issue).
  • Naming convention is followed (do please check it especially when you created new plugins, commands and resources).
  • Unit tests have been added for the changes (if applicable).
  • Breaking changes have been documented (or no breaking changes introduced in this PR).

@btea btea requested a review from jikkai as a code owner February 21, 2025 14:25
Copy link

github-actions bot commented Feb 21, 2025

View Deployment

📑 Examples 📚 Storybook
🔗 Preview link 🔗 Preview link

@wzhudev
Copy link
Member

wzhudev commented Feb 24, 2025

Please take a look into this @jikkai. Are there any other lifecycle scritps we should keep?

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.

2 participants