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

added custom config dir argument #1

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

Conversation

flying-sheep
Copy link

to allow package maintainers to create and update global configs

@jcb91
Copy link
Member

jcb91 commented Jul 30, 2016

I'm afraid that this isn't really an appropriate place to put this functionality, as the _compat modules are intended to be an exact API copy of notebook 4.2. As a result, this will have to go into downstream functions/classes...

@flying-sheep
Copy link
Author

this file uses the functions.

what other functions can it use that allow configuring the config file location?

@jcb91
Copy link
Member

jcb91 commented Jul 31, 2016

Well, there are no ready-made functions, I guess, which I admit is frustrating. Might be worth submitting an issue/PR to upstream notebook?

But, the use case you point to in src/jupyter_contrib_nbextensions/install.py would use this only as part of calls to _get_config_dir (which is easily wrapped), toggle_serverextension_python (which is easily mimicked), install_nbextension_python (which doesn't actually write to any config anyway), and uninstall_nbextension_python (which, as it only seeks to remove the nbextension's require url from the config, could be ignored in our case, or wrapped). I'll wrap up a PR to src/jupyter_contrib_nbextensions/install.py to show what I mean...

@flying-sheep
Copy link
Author

OK, please cc me 😃

@flying-sheep
Copy link
Author

hi @jcb91, any news?

@jcb91
Copy link
Member

jcb91 commented Aug 17, 2016

apologies @flying-sheep I've been a bit preoccupied by house repairs for a couple of weeks, and in the last week my macbook's decided to throw a fit and use 80% cu time on kernel_task for no reason (joy!).

@flying-sheep
Copy link
Author

no problem, good luck with that!

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