Skip to content

Conversation

blink1073
Copy link
Contributor

Make it easier for subclasses to provide configurability and update docs.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.06%) to 75.598% when pulling 4880921 on cleanup-configurability into 8ff7b0e on master.

@coveralls
Copy link

coveralls commented May 10, 2019

Coverage Status

Coverage decreased (-0.06%) to 75.598% when pulling 6d03d2a on cleanup-configurability into 8ff7b0e on master.

@blink1073
Copy link
Contributor Author

cc @thomasjm

@blink1073 blink1073 merged commit 8d6730b into master May 10, 2019
-------------
``Metakernel`` subclasses can be configured by the user by adding a ``metakernel_config.py`` file to their
``jupyter`` config path. The base ``MetaKernel`` class offers ``plot_settings`` as a configurable trait.
``Metakernel`` implementations can subclasses can be configured by the user. The

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good @blink1073! Noticed a couple typos here, should probably change

implementations can subclasses can be configured by the user -> subclasses can be configured by the user
Subclasses can defined -> Subclasses can define

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cheers, fixed.

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