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

add Application.show_config[_json] #341

Merged
merged 5 commits into from
May 4, 2017
Merged

Conversation

minrk
Copy link
Member

@minrk minrk commented Nov 1, 2016

setting this value will skip Application start and show loaded config on stdout, instead.

if show_config_json is given, it will be json-formatted.

  • tests

Applications can inherit the default flags from traitlets.config.application.default_flags,
but this can be triggered immediately with:

any-jupyter-command --Application.show_config=True

cc @rgbkrk

@rgbkrk
Copy link
Member

rgbkrk commented Nov 1, 2016

Yay, this is great! I'll use this for smoke testing a Docker image before promoting to production.

@minrk minrk added this to the 5.0 milestone Dec 21, 2016
@ankostis
Copy link
Contributor

ankostis commented Jan 31, 2017

But why stop the app?
The option may just dump the configs in the logger (i.e. at INFO level), and continue normally.

This option is useful for every combination sub-commands and it was one of the first "extensions" I had to built my self.

@rgbkrk
Copy link
Member

rgbkrk commented Jan 31, 2017

The reason I was hoping for something like this is to be able to run validation tests against Jupyter installations. I do want the application to stop after exiting.

minrk added 3 commits May 3, 2017 16:05
setting this value will skip Application start and show loaded config on stdout, instead.

if show_config[_json is given, it will be json-formatted
@minrk minrk changed the title [WIP] add Application.show_config[_json] add Application.show_config[_json] May 4, 2017
@minrk minrk merged commit 83c87ac into ipython:master May 4, 2017
@minrk minrk deleted the showconfig branch May 4, 2017 09:57
@Carreau Carreau added 5.0-re-review Need to re-review for potential API impact changes. 5.0-minor rereviewed, minor change need to be put in changelog. and removed 5.0-re-review Need to re-review for potential API impact changes. labels Jun 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
5.0-minor rereviewed, minor change need to be put in changelog.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants