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 plugin support to the installer #127

Merged
merged 6 commits into from Aug 13, 2018
Merged

Conversation

yuvipanda
Copy link
Collaborator

@yuvipanda yuvipanda commented Aug 11, 2018

TLJH is a great base to build 'stacks' on top of
for various use cases. These 'stacks' should be built by
people who are domain experts in their fields, and easily
updateable with new TLJH versions. Extension points need
to be very clearly defined & evolvable, so we can modify
TLJH without fear of breaking everything.

pluggy is the plugin
mechanism for pytest spun out into its own library,
and fits our requirements well.

There is an experimental pangeo stack in progress at
https://github.com/yuvipanda/tljh-pangeo for an example
of how this would work

  • Add / update documentation
  • Add tests

TLJH is a great base to build 'stacks' on top of
for various use cases. These 'stacks' should be built by
people who are domain experts in their fields, and easily
updateable with new TLJH versions. Extension points need
to be very clearly defined & evolvable, so we can modify
TLJH without fear of breaking everything.

[pluggy](https://pluggy.readthedocs.io/) is the plugin
mechanism for pytest spun out into its own library,
and fits our requirements well.

There is an experimental pangeo stack in progress at
https://github.com/yuvipanda/tljh-pangeo for an example
of how this would work
This will lead to tests reported as passing even when they are actually
failing.
I guess sl is *technically* a game. I'm sure there was a long
winded argument about where it should be in a mailing list
somewhere.
@yuvipanda
Copy link
Collaborator Author

Documentation needs two parts:

  1. Specifying --plugin to the installer
  2. How to write a plugin

Will have minimal skeletons of these up before merging.

@yuvipanda
Copy link
Collaborator Author

Ok, somewhat reasonable documentation exists for this now! #133 and #132 would make things nicer, but this is good to go! <3

@yuvipanda yuvipanda merged commit e38628b into jupyterhub:master Aug 13, 2018
@yuvipanda yuvipanda mentioned this pull request Aug 13, 2018
2 tasks
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.

None yet

1 participant