Skip to content

docs: correct futures example comments - #249

Open
nightcityblade wants to merge 1 commit into
dask:mainfrom
nightcityblade:fix/issue-247
Open

docs: correct futures example comments#249
nightcityblade wants to merge 1 commit into
dask:mainfrom
nightcityblade:fix/issue-247

Conversation

@nightcityblade

Copy link
Copy Markdown

Fixes #247

Correct the inline descriptions for the double and add futures so they match the functions submitted in the example.

Validation:

  • Parsed and validated futures.ipynb with nbformat
  • pre-commit run --files futures.ipynb

@review-notebook-app

Copy link
Copy Markdown

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@nightcityblade

Copy link
Copy Markdown
Author

CI note: the failures are unrelated to this notebook-comment-only diff. The build-and-deploy and update-dependencies jobs are blocked by deprecated actions/cache v2 usage; pre-commit failed while GitHub cache services were unavailable; and the repo2docker build cannot solve the existing pinned Binder environment. Recent main-branch runs show the same workflow failures.

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.

Quickfix for copy and paste bug in futures.ipynb comments

1 participant