-
-
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(typings): change export interface to class definition #2293
Changes from 3 commits
9081d66
f863f52
da93979
1b9954e
c0d53fe
8e26715
4110d28
2ba654d
65c869b
af6a958
3d2fc1b
5effde9
1023621
99b1439
b8ed59c
0a61275
fd05cae
62bd147
e16f76d
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -854,19 +854,18 @@ declare module 'egg' { | |
* Singleton instance in Agent Worker, extend {@link EggApplication} | ||
*/ | ||
class Agent extends EggApplication { | ||
_wrapMessenger(): void | ||
} | ||
|
||
export interface ClusterOptions { | ||
framework: string // specify framework that can be absolute path or npm package | ||
baseDir: string // directory of application, default to `process.cwd()` | ||
plugins: object // customized plugins, for unittest | ||
workers: number // numbers of app workers, default to `os.cpus().length` | ||
port: number // listening port, default to 7001(http) or 8443(https) | ||
https: boolean // https or not | ||
key: string //ssl key | ||
cert: string // ssl cert | ||
typescript: boolean | ||
framework: string; // specify framework that can be absolute path or npm package | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. framework / baseDir / workers 都是可选的 |
||
baseDir: string; // directory of application, default to `process.cwd()` | ||
plugins: object; // customized plugins, for unittest | ||
workers: number; // numbers of app workers, default to `os.cpus().length` | ||
port: number; // listening port, default to 7001(http) or 8443(https) | ||
https: boolean; // https or not | ||
key: string; //ssl key | ||
cert: string; // ssl cert | ||
// typescript: boolean; | ||
} | ||
|
||
export function startCluster(options: ClusterOptions, callback: () => any): void | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 一样,加一下 There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. void 后面也顺手加个 |
||
|
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.
这样的注释,在 vscode 那边能看到提示么?
是不是要用 jsdoc 的方式来做注释?
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.
如果是 vsc ,鼠标 ctrl+hover时 可以显示注释内容的