8000 Fix new typings failures by bryevdv · Pull Request #11971 · bokeh/bokeh · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@bryevdv
Copy link
Member
@bryevdv bryevdv commented Jan 24, 2022

Not sure what test dep changed but there are several new flake8 errors regarding files under typings cc @mattpap

@mattpap
Copy link
Contributor
mattpap commented Jan 24, 2022

@bryevdv, what were the errors? I see nothing for 0.920 and the most recent 0.931.

@mattpap
Copy link
Contributor
mattpap commented Jan 24, 2022

List[T] and list[T] types are equivalent. We also specify language version 3.8, so mypy should not promote the newer approach (i.e. list[T]).

@mattpap
Copy link
Contributor
mattpap commented Jan 24, 2022

I should have read the comment, it's flake not mypy. Still, I updated to flake8 4.0 and there are no errors for me.

@bryevdv bryevdv force-pushed the bryanv/fix_typings branch from 6391861 to 922eb6e Compare January 24, 2022 21:33
@bryevdv
Copy link
Member Author
bryevdv commented Jan 24, 2022

@mattpap yah I'm not sure, I did also update to Python 3.9.9 in a new env maybe that's it 🤷 In any case I think this CI run should pass

@bryevdv bryevdv merged commit 1ebb703 into branch-3.0 Jan 24, 2022
@bryevdv bryevdv deleted the bryanv/fix_typings branch January 24, 2022 22:19
ianthomas23 pushed a commit that referenced this pull request May 12, 2022
@ianthomas23 ianthomas23 modified the milestones: 3.0, 2.4.3 May 12, 2022
ianthomas23 added a commit that referenced this pull request May 13, 2022
* Use weekly channel for gmap plots (#12113)

* Add DatetimeRangeSlider (#12034)

* Add DatetimeRangeSlider

* Add tests

* Add docs

* Simplify EqHistColorMapper calculations (#12084)

* Simplify EqHistColorMapper calculations

* Add numerical unit tests

* Better numerical unit tests

* Support rescale_discrete_levels in EqHistColorMapper (#12121)

* Drop baseline testing on MacOS and Windows (part of #11906)

* Try more of #11906

* Pin to django 3.x (#11867)

* Fix new typings failures (#11971)

* 12093 Update union syntax (#12094)

* Update union syntax under typings directory

* Update Sequence and Callable imports

* Force webgl backend when requested for testing (#11823)

* Force webgl backend when requested for testing

webgl backend normally falls back to canvas rendering, but that's
undesirable when testing specifically webgl support. For testing
the fallback mechanism, use `settings.force_webgl = false`.

* Install a specific version of chromium on linux

* Python 3.7 specific mypy fixes

* isort fix

* remove unused ignores

* more mypy

* yet more mypy

* re-skip previously skipped SVG test

* Clarify RenderRoot for components (#12036)

* Clarify RenderRoot for components

* unrelated typo fix

* Fix color function call (#11751)

* Fix typo in git.py (#12106)

verison -> version

* Fixed typo: s/server/serve/ (#12051)

* Add docs_toc block (#11989)

* Fix
ABF0
ed xwheel and xzoom tools with hard bounds (#11832) (#11834)

* Change DatetimeRangeSlider tests to fit older testing framework

* Fix webgl dashed line joins with butt end caps (#11814)

* Fix webgl dashed line joins with butt end caps

* Update baseline images

* Correctly handle odd-length dash patterns in webgl (#11819)

* Correctly render straight webgl lines with bevel joins (#12065)

* Limit tool-related tap events to the frame (#11938)

* 11965 fixes that null value with unspecified nan_format crashes table rendering (#12098)

* #11965: fixed error when trying to show null values in table columns using ScientificFormatter

* #11965: changed string to double quotes

* Squashed commits:

[4355e9165] 11965: removed whitespaces (+1 squashed commits)

Squashed commits:

[46984940d] #11965: moved nan_format to StringFormatter and made non-nullable

* #11965: changed default nan_format to NaN

* #11965: default nan/null format for all string derived cellformatters set to -

Co-authored-by: Harm Buisman <h.buisman@iknl.nl>

Co-authored-by: Bryan Van de Ven <bryan@bokeh.org>
Co-authored-by: Jada Lilleboe <82007190+jadalilleboe@users.noreply.github.com>
Co-authored-by: Mateusz Paprocki <mattpap@gmail.com>
Co-authored-by: g-parki <61096711+g-parki@users.noreply.github.com>
Co-authored-by: Ikko Ashimine <eltociear@gmail.com>
Co-authored-by: Rick van Hattem <wolph@wol.ph>
Co-authored-by: Timo Cornelius Metzger <39711796+tcmetzger@users.noreply.github.com>
Co-authored-by: Florent <florentbr@gmail.com>
Co-authored-by: harmbuisman <harmbuisman@gmail.com>
Co-authored-by: Harm Buisman <h.buisman@iknl.nl>
@github-actions
Copy link

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.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 26, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

0