-
-
Notifications
You must be signed in to change notification settings - Fork 403
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
Dropping support for Bokeh 2 #5891
Conversation
Codecov Report
@@ Coverage Diff @@
## main #5891 +/- ##
==========================================
+ Coverage 88.41% 88.56% +0.15%
==========================================
Files 312 313 +1
Lines 64953 65069 +116
==========================================
+ Hits 57427 57628 +201
+ Misses 7526 7441 -85
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 35 files with indirect coverage changes 📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
9190f06
to
d2fd3e4
Compare
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.
Great, thank you! All the changes seem straightforward and uncontroversial. LGTM
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
#5751 removed Bokeh 2 from the test suite.
This PR continues it and removes support from the code base.
The sorting of import will be done in another PR, where I will enable automatic linting for it.