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

Fix metadata #138

Merged
merged 3 commits into from
Apr 11, 2023
Merged

Fix metadata #138

merged 3 commits into from
Apr 11, 2023

Conversation

hbcarlos
Copy link
Member

@hbcarlos hbcarlos commented Apr 11, 2023

Updates jupyter_ydoc to fix an issue with the YNotebook. See jupyter-server/jupyter_ydoc#158

Follow up #121 :

  • Use a lock to ignore self-updates
  • Fixes a formatting error

@hbcarlos hbcarlos added the bug Something isn't working label Apr 11, 2023
@hbcarlos hbcarlos requested a review from fcollonval April 11, 2023 16:00
@hbcarlos hbcarlos self-assigned this Apr 11, 2023
@github-actions
Copy link
Contributor

Binder 👈 Launch a Binder on branch hbcarlos/jupyter_collaboration/fix-metadata

@codecov-commenter
Copy link

codecov-commenter commented Apr 11, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (ba6d756) 0.00% compared to head (00a3254) 0.00%.

Additional details and impacted files
@@          Coverage Diff          @@
##            main    #138   +/-   ##
=====================================
  Coverage   0.00%   0.00%           
=====================================
  Files          7       7           
  Lines        418     408   -10     
=====================================
+ Misses       418     408   -10     
Impacted Files Coverage Δ
jupyter_collaboration/loaders.py 0.00% <0.00%> (ø)
jupyter_collaboration/rooms.py 0.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@hbcarlos hbcarlos merged commit 2e5a6cc into jupyterlab:main Apr 11, 2023
@hbcarlos hbcarlos deleted the fix-metadata branch April 11, 2023 16:16
@davidbrochart
Copy link
Collaborator

@hbcarlos I would appreciate if you give me a bit of time to be able to review, instead of merging your own PR.

@fcollonval
Copy link
Member

@davidbrochart @hbcarlos what do you think of adding a rule requiring an approved review to allow merge (I guess we should also do the same on jupyter_ydoc)?

@davidbrochart
Copy link
Collaborator

Yes I think it would be useful, here and in jupyter-ydoc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants