-
-
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
fix: try to use custom function name first #2161
Conversation
@dead-horse 好了 |
|
||
const fs = require('mz/fs'); | ||
|
||
module.exports = function* () { |
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.
这里我改成 aa
8a3f390
to
d4e2c66
Compare
|
||
const fs = require('mz/fs'); | ||
|
||
module.exports = async (ctx) => { |
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.
async ctx =>
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.
这个 lint 不会报?
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.
fixtures �ignore 了吧
Codecov Report
@@ Coverage Diff @@
## master #2161 +/- ##
=======================================
Coverage 99.59% 99.59%
=======================================
Files 29 29
Lines 749 749
=======================================
Hits 746 746
Misses 3 3
Continue to review full report at Codecov.
|
fixed |
fix: try to use custom function name first (#2161)
pick from #2156
Checklist
npm test
passesAffected core subsystem(s)
Description of change