-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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: Release the first stable version 🥚 👏👏👏 #605
Conversation
@popomore, thanks for your PR! By analyzing the history of the files in this pull request, we identified @atian25, @fengmk2 and @dead-horse to be potential reviewers. |
👏👏👏 |
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.
👍👍👍
Codecov Report
@@ Coverage Diff @@
## master #605 +/- ##
=====================================
Coverage 100% 100%
=====================================
Files 26 26
Lines 633 633
=====================================
Hits 633 633 Continue to review full report at Codecov.
|
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.
👏 👏 👏
@popomore Please merge and release! |
覆盖率降低和 egg-bin 有关系? |
@popomore 换回去 2.x 跑跑看? |
|
有可能是这个原因, https://github.com/eggjs/egg-bin/blob/master/lib/cmd/cov.js#L59 上面复制了,下面才赋值,我发个 PR 修复下 |
991eab3
to
1345abf
Compare
win 又卡在了 -.- |
覆盖率好像解决了 |
Checklist
npm test
passesAffected core subsystem(s)
Description of change
Closes #5