Skip to content

Add some basic examples to doc strings on dask.bag all, any, count methods#7630

Merged
jrbourbeau merged 3 commits intodask:mainfrom
ndanielsen:add-some-docs-to-dask-bags
Sep 30, 2021
Merged

Add some basic examples to doc strings on dask.bag all, any, count methods#7630
jrbourbeau merged 3 commits intodask:mainfrom
ndanielsen:add-some-docs-to-dask-bags

Conversation

@ndanielsen
Copy link
Copy Markdown
Contributor

Found the API documentation was lacking basic examples on these methods, so figured this would be helpful to others.

  • Closes #xxxx
  • Tests added / passed
  • Passes black dask / flake8 dask / isort dask

@GenevieveBuckley
Copy link
Copy Markdown
Contributor

I like these, I think it's helpful to have them here.

@quasiben
Copy link
Copy Markdown
Member

quasiben commented May 6, 2021

Looks like there are still some linting issues. If you run black dask/bag/ that should take care of it. We have docs on getting set up with black/pre-commit here

@GPUtester
Copy link
Copy Markdown
Collaborator

Can one of the admins verify this patch?

@jrbourbeau
Copy link
Copy Markdown
Member

add to allowlist

Copy link
Copy Markdown
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

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

Thanks @ndanielsen! I merged main and ran our pre-commit hooks (hope that's okay). This looks great, will merge once CI finishes up

Also, I noticed this is your first code contribution to this repository. Welcome!

@ndanielsen
Copy link
Copy Markdown
Contributor Author

Thanks @jrbourbeau feels good to contribute here!

@jrbourbeau jrbourbeau merged commit bbda552 into dask:main Sep 30, 2021
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.

5 participants