-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
Fix typos in jupyter-collaboration
-missing error message
#16436
Conversation
Thanks for making a pull request to jupyterlab! |
Thanks for submitting your first pull request! You are awesome! 🤗 |
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.
Thank you!
To fix this, either: | ||
1) install the extension `jupyter_collaboration`; for example: `python -m pip install jupyter_collaboration` | ||
1) install the extension `jupyter-collaboration`; for example: `python -m pip install jupyter-collaboration` |
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.
👍 indeed, this is preferred. pip
does not care, but conda does not normalize, and the feedstock uses -
over _
(https://github.com/conda-forge/jupyter-collaboration-feedstock)
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.
yes sorry! I copy-pasted into conda and had it fail which is why I opened this PR
In future you can ask the bot to do so like: @meeseeksdev tag bug |
@meeseeksdev please backport to 4.2.x |
…ssing error message
…r message (#16440) Co-authored-by: Krishan Bhasin <164889026+krishanbhasin-px@users.noreply.github.com>
References
N/A
Code changes
Update the content of the
log.critical
message aboutjupyter-collaboration
being missingUser-facing changes
Jupyter
is spelled correctlyjupyter-collaboration
can now be copy-pasted by users to fix the issue (prior usage ofjupyter_collaboration
was different to the conda and pip package names)Backwards-incompatible changes
N/A
Notes
CI is failing with
but I lack the permissions to do so