Skip to content

Added the ability to commit changes with nb #3

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

takotab
Copy link
Contributor

@takotab takotab commented Mar 29, 2020

I really like your save functions they have already save me twice. That why I thought maybe this function that I use would also helpful.

I can imaging if you would like to remove it from the requirements since it is only useful for dev and not really for usage. But that's up to you. There is more here:

https://forums.fast.ai/t/quickly-add-changes-to-git/64152

@oguiza
Copy link
Contributor

oguiza commented Apr 6, 2020

I think your proposal is very interesting. Thanks for sharing!
I'd be interested in adding it if we can automatically get the current file name, which is difficult to get. In this way you would run the nb and get it saved, .py files created and sent to git, which I think would be great!

@oguiza
Copy link
Contributor

oguiza commented Apr 6, 2020

Actually, I know you can get your current nb name by running

%%javascript
IPython.notebook.kernel.execute('nb_name = "' + IPython.notebook.notebook_name + '"')

in a separate cell. Maybe you can integrate it with your function.

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.

2 participants