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

Final pass before 5.0 release #996

Merged
merged 12 commits into from
Oct 8, 2023

Conversation

krassowski
Copy link
Member

  • Bump dependent packages to final
  • Check if we can fix the remaining tests, if not ignore to make catching regressions easier
  • Fixed integration tests
    • Completes In R Magics - there was a problem with cell being out of view in a long test notebook
    • Completes In Strings Or Python Dictionaries - fixed by adding 2 seconds delay, may require future investigation
    • Hover Triggers Automatically - fixed by increasing retrials of wiggling simulation
    • Update hover after character deletion - fixed by increasing retrials of wiggling simulation
  • Integration tests skipped due to known upstream issues:
  • Integration tests skipped as passing locally (help wanted):
    • Columns Can Be Hidden - passes locally; also tested in Firefox and Chrome works ok
    • Hover works in foreign code (javascript) - passes locally but flaky
    • Diagnostics Panel Works After Removing Foreign Document - passes locally
    • Python Dotted this highlights a genuine problem with dotted settings, now documented.

Many of the tests will not be fixed before JupyterLab 4.1 gets released because of how unreliable is the workaround of getting the adapter.

@github-actions
Copy link

github-actions bot commented Oct 7, 2023

Binder 👈 Launch a binder notebook on branch krassowski/jupyterlab-lsp/remove-rc-suffixes

@krassowski krassowski merged commit 7522350 into jupyter-lsp:main Oct 8, 2023
17 of 23 checks passed
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.

1 participant