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: correct types for domain unionWith #9131

Merged
merged 2 commits into from
Oct 9, 2023
Merged

Conversation

kanitw
Copy link
Member

@kanitw kanitw commented Oct 6, 2023

Fix #9130

Fix in correct types as discussed in #9130

@kanitw kanitw requested a review from a team October 6, 2023 23:59
@kanitw kanitw merged commit 6f66759 into main Oct 9, 2023
9 checks passed
@kanitw kanitw deleted the kw/9130-unionWith-type branch October 9, 2023 17:32
BradyJ27 pushed a commit to BradyJ27/vega-lite that referenced this pull request Oct 19, 2023
* fix: correct types for domain unionWith

* chore: update schema [CI]

---------

Co-authored-by: GitHub Actions Bot <vega-actions-bot@users.noreply.github.com>
alliefeldman pushed a commit to alliefeldman/vega-lite that referenced this pull request Nov 2, 2023
* fix: correct types for domain unionWith

* chore: update schema [CI]

---------

Co-authored-by: GitHub Actions Bot <vega-actions-bot@users.noreply.github.com>
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.

Type for scale domain.unionWith is too strict
2 participants