Skip to content
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

Try to get nbdime used by github #243

Closed
vidartf opened this issue Jan 17, 2017 · 12 comments
Closed

Try to get nbdime used by github #243

vidartf opened this issue Jan 17, 2017 · 12 comments

Comments

@vidartf
Copy link
Collaborator

vidartf commented Jan 17, 2017

No description provided.

@vidartf vidartf changed the title Try to get nbdime used by gihub Try to get nbdime used by github Jan 17, 2017
@willingc
Copy link
Member

willingc commented Mar 1, 2017

Hi @bkeepers,

I enjoyed meeting you at the wont_fix unconference at GH a couple of weeks ago. It was a lovely event, and I like GH's direction with opensource.guide (I shared with hundreds of students at Hack Illinois this past weekend).

I wanted to follow up on a conversation with Lee and others about possibly seeing if GitHub could incorporate nbdime for diffing Jupyter/IPython notebooks that GitHub renders. Would you mind pointing us in the right direction of who/where we should formally share this request with at GH?

Thanks 😄

Carol

cc/ @minrk @vidartf

@sixers
Copy link

sixers commented Mar 6, 2017

Is it possible to create a JavaScript extension for Github? One of the ways to do this is to fetch diff data via API, and just provide a GUI on top of Github. This way we're not dependent on Github implementing this feature, and we can also support multiple vendors, e.g. Gitlab.

@bkeepers
Copy link

bkeepers commented Mar 6, 2017

Hey @willingc, it was great to meet you too!

support@github.com is the best place to send feature requests. Our support team does a great job of cataloging feature requests so our engineering teams can see them.

@willingc
Copy link
Member

willingc commented Mar 6, 2017

Thanks @bkeepers for the direction. We'll send it over formally (though I think it has been raised in some informal talks already).

@Analect
Copy link

Analect commented Mar 14, 2017

For what it's worth, I added a comment to this thread over on Gitlab, since there are efforts underway there to try to get a basic notebook viewer implemented. It could be that @vidartf or @sixers above could enhance the conversation over there and discuss the possibilities of gitlab being able to leverage the nbdime accomplishments.

@Analect
Copy link

Analect commented Mar 28, 2017

FYI. Basic implementation for viewing notebooks has been added to master on Gitlab here. Still may be worth engaging with them to get nbdime working on that platform.

@orionr
Copy link

orionr commented Jan 17, 2018

Any progress on this? Having the ability to diff ipynb files in GitHub PRs would be huge.

@joelostblom
Copy link
Contributor

There is now a GitLab issue for adding rich diff support to notebooks.

@amit1rrr
Copy link

amit1rrr commented Oct 29, 2018

I built ReviewNB GitHub App for Jupyter notebook code reviews. More about it here.

It renders visual diffs for notebook commits and pull requests & lets you write comments to discuss changes. Comments are ultimately posted back to GitHub PR for storage.

screenshot 2018-10-23 at 12 44 04 pm

@gwincr11
Copy link
Contributor

gwincr11 commented Mar 1, 2023

https://github.blog/changelog/2023-03-01-feature-preview-rich-jupyter-notebook-diffs/

@minrk
Copy link
Member

minrk commented Mar 8, 2023

Huzzah!

@minrk minrk closed this as completed Mar 8, 2023
@gwincr11
Copy link
Contributor

gwincr11 commented Mar 8, 2023

Congrats and thanks for building a cool tool. Hope to meet you at Jupyter Conf.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

10 participants