8000 WIP An initial implementation of cells by jasongrout · Pull Request #3 · jupyter/jupyter-js-cells · GitHub
[go: up one dir, main page]

Skip to content
This repository was archived by the owner on Oct 2, 2023. It is now read-only.
8000

WIP An initial implementation of cells #3

Closed
wants to merge 8 commits into from
Closed

WIP An initial implementation of cells #3

wants to merge 8 commits into from

Conversation

jasongrout
Copy link
Member

import {
IInputAreaViewModel
} from 'jupyter-js-input-area';
import {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing blank line.

@blink1073
Copy link
Contributor

Ran the example, looks good, I'd recommend adding this:

diff --git a/package.json b/package.json
index 0c28fd5..c264851 100644
--- a/package.json
+++ b/package.json
@@ -58,6 +58,11 @@
     "lib/*.js",
     "lib/*.d.ts"
   ],
+  "system": {
+    "npmIgnore": [
+      "devDependencies"
+    ]
+  },
   "author": "Project Jupyter",
   "license": "BSD-3-Clause",
   "bugs": {

@jasongrout jasongrout changed the title An initial implementation of cells WIP An initial implementation of cells Dec 11, 2015
@jasongrout
Copy link
Member Author

@blink1073 - Thanks for looking at this. I forgot to label this WIP. It's definitely not ready for scrutiny about documentation, whitespace, etc. I'm moving on to the notebook repo, which will test the design of this. When the basic system works, I'm coming back to go over the documentation, whitespace, etc.

@blink1073
Copy link
A0DC
Contributor

Ha, fair enough.

@jasongrout
Copy link
Member Author

Superseded by #4

@jasongrout jasongrout closed this Jan 9, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants
0