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

Use Keyword-only arguments #2814

Merged
merged 3 commits into from
Jun 30, 2019
Merged

Use Keyword-only arguments #2814

merged 3 commits into from
Jun 30, 2019

Conversation

mrocklin
Copy link
Member

Previously for functions with both *args and **kwargs inputs we
often explicitly popped off values explicitly. Now that we no longer
support Python 2 we can use these as keyword arguments directly.

Previously for functions with both `*args` and `**kwargs` inputs we
often explicitly popped off values explicitly.  Now that we no longer
support Python 2 we can use these as keyword arguments directly.
@mrocklin mrocklin merged commit 9aa0ea6 into dask:master Jun 30, 2019
@mrocklin mrocklin deleted the kw-only-arguments branch June 30, 2019 20:02
muammar added a commit to muammar/distributed that referenced this pull request Jul 18, 2019
* upstream/master: (33 commits)
  SpecCluster: move init logic into start (dask#2850)
  Dont reuse closed worker in get_worker (dask#2841)
  Add alternative SSHCluster implementation (dask#2827)
  Extend prometheus metrics endpoint (dask#2792) (dask#2833)
  Include type name in SpecCluster repr (dask#2834)
  Don't make False add-keys report to scheduler (dask#2421)
  Add Nanny to worker docs (dask#2826)
  Respect security configuration in LocalCluster (dask#2822)
  bump version to 2.1.0
  Fix typo that prevented error message (dask#2825)
  Remove dask-mpi (dask#2824)
  Updates to use update_graph in task journey docs (dask#2821)
  Fix Client repr with memory_info=None (dask#2816)
  Fix case where key, rather than TaskState, could end up in ts.waiting_on (dask#2819)
  Use Keyword-only arguments (dask#2814)
  Relax check for worker references in cluster context manager (dask#2813)
  Add HTTPS support for the dashboard (dask#2812)
  CLN: Use dask.utils.format_bytes (dask#2810)
  bump version to 2.0.1
  Add python_requires entry to setup.py (dask#2807)
  ...
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.

None yet

1 participant