Skip to content

Comments

ci: Bump vl-convert-python to 1.7.0#3633

Merged
dangotbanned merged 1 commit intomainfrom
bump-fix-vl-convert-1.7.0
Oct 6, 2024
Merged

ci: Bump vl-convert-python to 1.7.0#3633
dangotbanned merged 1 commit intomainfrom
bump-fix-vl-convert-1.7.0

Conversation

@dangotbanned
Copy link
Member

@dangotbanned dangotbanned commented Oct 6, 2024

Related

I ran into this on another branch, good news is the new stubs are working correctly:

(doc) PS C:\Users\danie\Documents\GitHub\altair> mypy tools
tools\generate_schema_wrapper.py:496: error: Argument 1 to "_vega_lite_props_only" has incompatible type
"dict[Literal['carbong10', 'carbong100', 'carbong90', 'carbonwhite', 'dark', 'excel', 'fivethirtyeight', 'ggplot2', 'googlecharts', 'latimes', 'powerbi', 'quartz', 'urbaninstitute', 'vox'], dict[str, Any]]"; expected
"dict[str, dict[str, Any]]"  [arg-type]
        themes = dict(_vega_lite_props_only(vlc.get_themes(), vl_props))

@dangotbanned dangotbanned added maintenance dependencies Pull requests that update a dependency file labels Oct 6, 2024
@dangotbanned dangotbanned enabled auto-merge (squash) October 6, 2024 17:00
@dangotbanned dangotbanned merged commit 02ad17d into main Oct 6, 2024
@dangotbanned dangotbanned deleted the bump-fix-vl-convert-1.7.0 branch October 6, 2024 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file maintenance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant