Skip to content

Conversation

@jodal
Copy link
Owner

@jodal jodal commented Jun 8, 2023

  • Make block and timeout arguments kwargs-only
  • Make ActorProxy and CallableProxy only accept kwargs

@codecov
Copy link

codecov bot commented Jun 8, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (1d4ba71) 92.11% compared to head (4fd0dbe) 92.11%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #198   +/-   ##
=======================================
  Coverage   92.11%   92.11%           
=======================================
  Files          13       13           
  Lines         558      558           
  Branches       96       96           
=======================================
  Hits          514      514           
  Misses         40       40           
  Partials        4        4           
Impacted Files Coverage Δ
src/pykka/_threading.py 92.98% <ø> (ø)
src/pykka/_future.py 100.00% <100.00%> (ø)
src/pykka/_proxy.py 96.42% <100.00%> (ø)
src/pykka/_ref.py 91.66% <100.00%> (ø)
src/pykka/_registry.py 95.58% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@jodal jodal merged commit d0f5154 into main Jun 8, 2023
@jodal jodal deleted the kwargs-only branch June 8, 2023 22:25
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.

2 participants