Skip to content

Commit

Permalink
Remove legacy "interpreter" plugin (#61263)
Browse files Browse the repository at this point in the history
* chore: πŸ€– delete legacy interpreter registries

* chore: πŸ€– completely delete legacy "interpreter" plugin

* chore: πŸ€– remove dependencies on legacy "interpreter" plugin

* chore: πŸ€– revert back Canvas global registries

* test: πŸ’ fix Lens test

Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
  • Loading branch information
streamich and elasticmachine authored May 5, 2020
1 parent cf64454 commit d77d2e4
Show file tree
Hide file tree
Showing 11 changed files with 3 additions and 262 deletions.
2 changes: 0 additions & 2 deletions src/legacy/core_plugins/interpreter/README.md

This file was deleted.

44 changes: 0 additions & 44 deletions src/legacy/core_plugins/interpreter/index.ts

This file was deleted.

52 changes: 0 additions & 52 deletions src/legacy/core_plugins/interpreter/init.ts

This file was deleted.

4 changes: 0 additions & 4 deletions src/legacy/core_plugins/interpreter/package.json

This file was deleted.

This file was deleted.

49 changes: 0 additions & 49 deletions src/legacy/core_plugins/interpreter/public/interpreter.ts

This file was deleted.

This file was deleted.

29 changes: 0 additions & 29 deletions src/legacy/core_plugins/interpreter/public/registries.ts

This file was deleted.

2 changes: 0 additions & 2 deletions x-pack/legacy/plugins/canvas/.storybook/storyshots.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -77,8 +77,6 @@ jest.mock('@elastic/eui/lib/services/accessibility/html_id_generator', () => {
};
});

jest.mock('plugins/interpreter/registries', () => ({}));

// Disabling this test due to https://github.com/elastic/eui/issues/2242
jest.mock(
'../public/components/workpad_header/share_menu/flyout/__examples__/share_website_flyout.stories',
Expand Down
4 changes: 2 additions & 2 deletions x-pack/legacy/plugins/canvas/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ Open your plugin's `index.js` file, and modify it to look something like this (b
export default function (kibana) {
return new kibana.Plugin({
// Tell Kibana that this plugin needs canvas and the Kibana interpreter
require: ['interpreter', 'canvas'],
require: ['canvas'],

// The name of your plugin. Make this whatever you want.
name: 'canvas_example',
Expand Down Expand Up @@ -132,7 +132,7 @@ In your plugin's root `index.js` file, modify the `kibana.Plugin` definition to
export default function (kibana) {
return new kibana.Plugin({
// Tell Kibana that this plugin needs canvas and the Kibana interpreter
require: ['interpreter', 'canvas'],
require: ['canvas'],

// The name of your plugin. Make this whatever you want.
name: 'canvas_example',
Expand Down
2 changes: 1 addition & 1 deletion x-pack/legacy/plugins/canvas/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ export function canvas(kibana) {
return new kibana.Plugin({
id: CANVAS_APP,
configPrefix: 'xpack.canvas',
require: ['kibana', 'elasticsearch', 'xpack_main', 'interpreter'],
require: ['kibana', 'elasticsearch', 'xpack_main'],
publicDir: resolve(__dirname, 'public'),
uiExports: {
app: {
Expand Down

0 comments on commit d77d2e4

Please sign in to comment.