-
Notifications
You must be signed in to change notification settings - Fork 12
Upgrade #231
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
Conversation
The 'community.general.yaml' callback plugin has been removed. The plugin has been superseded by the option `result_format=yaml`
6428dfe to
849ffbc
Compare
f8dd1be to
83dde11
Compare
fcd073c to
bf7af72
Compare
33eba17 to
f8446d6
Compare
|
Please update PR description |
|
@trungleduc They are ok with only support sqlite. |
| from traitlets import Unicode | ||
|
|
||
| from .permissions import Permissions, PermissionsHandler, PermissionsAPIHandler | ||
| DB_URL = "sqlite:///tljh_plasma.sqlite" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
let's make it configurable via an environment variable with the default value being the current one
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This PR cleans up the code, upgrades to JupyterHub 5, and converts to service.