Fix Client submit type hint docstring

[nadzhou]

Closes #xxxx

• Tests added / passed
• Passes pixi run lint

Fixes #8429

[nadzhou]

@fjetter I made the changes as per request but lint is throwing me errors now:

codespell................................................................Passed
  mypy.....................................................................Failed
  - hook id: mypy
  - exit code: 1
  
  distributed/client.py:2020: error: Arguments not allowed after ParamSpec.args  [valid-type]
  distributed/shuffle/tests/test_merge.py:27: error: Unused "type: ignore" comment  [unused-ignore]
  distributed/shuffle/tests/test_shuffle.py:80: error: Unused "type: ignore" comment  [unused-ignore]

[github-actions]

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

    40 files  ±0      40 suites  ±0   14h 33m 24s ⏱️ - 5m 36s
 4 160 tests ±0   3 980 ✅ ±0    178 💤 ±0  2 ❌ ±0 
80 961 runs  +1  76 718 ✅ ±0  4 241 💤 +1  2 ❌ ±0 

For more details on these failures, see this check.

Results for commit 335659d. ± Comparison against base commit 0a341df.

♻️ This comment has been updated with latest results.

[jonasdedden]

Sorry for giving wrong suggestions in the issue some years ago, back then I lacked a bit of understanding around ParamSpec. I believe because submit and map introduce their own keyword parameters, one can't differentiate them from the Callable's ParamSpec.

That's why I mypy is complaining about the use of ParamSpec with other keyword arguments (P.args and P.kwargs have to be used directly besides each other) - I believe the only way put here is to just omit typehinting of args and kwargs. Then I'd say it would be sensible to reduce the scope of this PR to just removing the wrong tuple typehint in the docstring.

Just FYI, there are ways to fully typehint this, but they would require breaking changes to the submit and map function signature, as far as I can see.

[nadzhou]

Just FYI, there are ways to fully typehint this, but they would require breaking changes to the submit and map function signature, as far as I can see.

Yeah, we can wrap all other input arguemnts to *args, for example.

[nadzhou]

@fjetter done now.

[nadzhou]

@jonasdedden I've updated the branch now.