test(egg-bin): prevent dependency install scripts - #6047
Conversation
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## next #6047 +/- ##
==========================================
- Coverage 83.08% 78.50% -4.58%
==========================================
Files 731 18 -713
Lines 22496 842 -21654
Branches 4530 227 -4303
==========================================
- Hits 18690 661 -18029
+ Misses 3299 153 -3146
+ Partials 507 28 -479 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
@eggjs/bindoes not define dependency install scripts. This change documents that behavior. It also adds a regression test forpreinstall,install, andpostinstall.Package managers do not need an install-script allowlist entry for
@eggjs/bin.The focused test and the full
@eggjs/bintest suite pass.Closes eggjs/bin#294.