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: first implement #1
Conversation
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.
+1
'use strict'; | ||
|
||
module.exports = { | ||
instrument(event, action) { |
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.
jsdoc
const ms = require('ms'); | ||
|
||
class Instrument { | ||
constructor(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.
要 extend BaseContextClass 么?
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 突然间想到不能只能 local 开启,instrument 方法应该一直存在才对,只是 prod 会将 instrument 设置为空函数 |
那我改下 |
Closes eggjs/egg#280