-
-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
diff-ability of notebooks #3065
Comments
Input prompts are output information, and they definitely belong in the cell data. For those who want to strip these for git reasons, it should be easy to write a git pre-commit hook that either just strips the prompts, or does a full
Where are you seeing this? It isn't true of input or markdown. |
yes, the commit hook solution is what I thought, but that's pretty techy for drive-by contributors. I see long lines in all markdown cells of a v3 notebooks, in the 'source' field. Is there something I should be doing |
Codemirror might be soft wrapping. |
Ah, I see - long lines that you type will be preserved (just like a regular text file). |
good, in that case that's exactly as it should be. thanks for the tips. |
I've had a few sessions of reading through notebooks containing lots of text
and sending PRs for fixes. There are a couple of snags i hit:
to changes in the "prompt_number" fields.
This necessitates individually filtering them out with "git add -p", which
gets tedious quickly.
I realize this is a constraint imposed by JSON, but it means that
deleting a spurious comman results in a diff for what could be 50
lines of text, as diff is line-based.
I can't suggest a solution that doesn't involve changing the nb format,
so I guess this is just fyi.
The text was updated successfully, but these errors were encountered: