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

adding a link generator binder #49

Merged
merged 1 commit into from
Jul 23, 2018
Merged

Conversation

choldgraf
Copy link
Member

@choldgraf choldgraf commented Jul 22, 2018

this riffs off of @yuvipanda 's "inferentialthinking" binder generator, but now for generic nbgitpuller links!

you can demo it here:

https://mybinder.org/v2/gh/choldgraf/gitautosync/link_gen?urlpath=apps/binder%2Flink_generator.ipynb

@ryanlovett
Copy link
Contributor

Great minds think alike, https://github.com/ryanlovett/gitpullapp. At the time however, @yuvipanda had wanted to keep the app and the nb extension separate. Yuvi, any new thoughts on this?

@choldgraf
Copy link
Member Author

ha, nice! I like yours better, shall I close this? (feel free to close if you like!) I can always make PRs to that repo.

@ryanlovett
Copy link
Contributor

I do think that it makes sense to put the app in this repo, but I'll defer to Yuvi.

@yuvipanda yuvipanda merged commit 6e6161b into jupyterhub:master Jul 23, 2018
@yuvipanda
Copy link
Contributor

Ah, I think this is ok. @ryanlovett I thought you meant putting it in the nbgitpuller extension itself, not this repo. I apologize for the misunderstanding! This is totally ok!

BTW, @choldgraf - My 'inferentialthinking.com link generator' is actually riffed off of Ryan's app!

@choldgraf
Copy link
Member Author

haha - then we have come full-circle

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

Successfully merging this pull request may close these issues.

3 participants