-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Add some types for interpter plugin registration #31600
Add some types for interpter plugin registration #31600
Conversation
💔 Build Failed |
643c81b
to
958530d
Compare
💔 Build Failed |
958530d
to
d757b85
Compare
💔 Build Failed |
@@ -30,7 +30,8 @@ describe('timeseriesFetcher', () => { | |||
end: 1528977600000, | |||
client: clientSpy, | |||
config: { | |||
get: () => 'myIndex' as any | |||
get: () => 'myIndex' as any, | |||
has: () => true |
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.
@stacey-gammon Do we need to mock this property if we never call it?
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.
It appears so @sqren:
unless I am missing something, I believe since the function expects the Config object, even if it's not used internally, because it's required on the config object, it has to exist.
💔 Build Failed |
1df28bf
to
8f95cd1
Compare
💔 Build Failed |
Failed on what looks like a ci issue:
jenkins, test this |
💔 Build Failed |
Pinging @elastic/kibana-platform |
💚 Build Succeeded |
f4772d3
to
9b5ef32
Compare
@joshdover - any more comments or is this good to merge? |
💔 Build Failed |
kibana.d.ts
Outdated
export type InitPluginFunction = (server: Server) => void; | ||
export interface UiExports { | ||
injectDefaultVars: (server: Server) => { [key: string]: any }; | ||
} |
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.
Can InitPluginFunction
and UiExports
be re-exported from LegacyKibanaPluginSpec
, since they're already included once there?
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.
yes that makes much more sense.
💚 Build Succeeded |
* Some basic types for top level legacy plugin registration * put typescript interface closer to implementation * Use default export, you have to * fix build and address code review comments
* Some basic types for top level legacy plugin registration * put typescript interface closer to implementation * Use default export, you have to * fix build and address code review comments
Some typings for top level legacy plugin registration, and use-age in interpreter plugin.