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

Sync selection_* docstrings with signatures #3001

Merged
merged 3 commits into from
Mar 29, 2023

Conversation

dpoznik
Copy link
Contributor

@dpoznik dpoznik commented Mar 28, 2023

Resolves #3000

altair/vegalite/v5/api.py Show resolved Hide resolved
altair/vegalite/v5/api.py Show resolved Hide resolved
altair/vegalite/v5/api.py Outdated Show resolved Hide resolved
@dpoznik dpoznik marked this pull request as ready for review March 28, 2023 19:15
Copy link
Contributor

@joelostblom joelostblom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is great, thank you for contributing @dpoznik ! And on your first day of using the library, that must be a record =) Just a few comments.

altair/vegalite/v5/api.py Show resolved Hide resolved
altair/vegalite/v5/api.py Outdated Show resolved Hide resolved
@dpoznik dpoznik changed the title Sync selection_* docstrings with parameters Sync selection_* docstrings with signatures Mar 28, 2023
@dpoznik dpoznik force-pushed the update-selection_interval-docstring branch from 2b94e60 to f770daa Compare March 28, 2023 21:53
Copy link
Contributor

@joelostblom joelostblom left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! Thanks again @dpoznik

@joelostblom joelostblom merged commit 5d8c41b into vega:master Mar 29, 2023
@dpoznik
Copy link
Contributor Author

dpoznik commented Mar 29, 2023

Looks great! Thanks again @dpoznik

Cool, happy to help. I'm really enjoying using the library!

@dpoznik dpoznik deleted the update-selection_interval-docstring branch March 29, 2023 00:15
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.

selection_interval docs describe unusable "fields" parameter
3 participants