-
Notifications
You must be signed in to change notification settings - Fork 54
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
feat: disable espower by default #196
Conversation
@eggjs/tsconfig compile to es2020 by default and espower-source don't support it
https://npmmirror.com/package/espower-source 已经失去维护,接下来会直接去掉它。 |
忘记 power-assert 吧,有 typings 提示的情况下,有 API 很多也不怕 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
v5 先默认改成 false,发个 minor。然后我再提交一个去掉 espower 发 major。 |
任何事情都是有代价,power-assert 理念很好,但是很难长期做下去。 |
Codecov ReportBase: 100.00% // Head: 100.00% // No change to project coverage 👍
Additional details and impacted files@@ Coverage Diff @@
## master #196 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 12 8 -4
Lines 93 60 -33
Branches 16 12 -4
=========================================
- Hits 93 60 -33 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
[skip ci] ## [5.7.0](v5.6.1...v5.7.0) (2022-12-18) ### Features * disable espower by default ([#196](#196)) ([5e35438](5e35438))
🎉 This PR is included in version 5.7.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
@eggjs/tsconfig compile to es2020 by default and espower-source don't support it
Checklist
npm test
passesAffected core subsystem(s)
Description of change