Skip to content

Commit

Permalink
Remove notebook output logging.
Browse files Browse the repository at this point in the history
Notebook output logging made it easier to test the logging console extension. I think it is too much noise for production, though, unless it is behind a option and turned off by default. See jupyterlab#7386
  • Loading branch information
jasongrout committed Oct 22, 2019
1 parent f34ca0d commit 0d4115d
Show file tree
Hide file tree
Showing 4 changed files with 1 addition and 61 deletions.
1 change: 0 additions & 1 deletion packages/logconsole-extension/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,6 @@
"@jupyterlab/mainmenu": "^2.0.0-alpha.1",
"@jupyterlab/notebook": "^2.0.0-alpha.1",
"@jupyterlab/rendermime": "^2.0.0-alpha.1",
"@jupyterlab/services": "^5.0.0-alpha.1",
"@jupyterlab/statusbar": "^2.0.0-alpha.1",
"@jupyterlab/ui-components": "^2.0.0-alpha.1",
"@phosphor/signaling": "^1.3.0",
Expand Down
4 changes: 1 addition & 3 deletions packages/logconsole-extension/src/index.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -274,6 +274,4 @@ function activateLogConsole(
return loggerRegistry;
}

// TODO: delete the nboutput widget, or at least make it a non-default option?
import { logNotebookOutput } from './nboutput';
export default [logConsolePlugin, logNotebookOutput];
export default [logConsolePlugin];
54 changes: 0 additions & 54 deletions packages/logconsole-extension/src/nboutput.ts

This file was deleted.

3 changes: 0 additions & 3 deletions packages/logconsole-extension/tsconfig.json
Original file line number Diff line number Diff line change
Expand Up @@ -27,9 +27,6 @@
{
"path": "../rendermime"
},
{
"path": "../services"
},
{
"path": "../statusbar"
},
Expand Down

0 comments on commit 0d4115d

Please sign in to comment.