Add typing annotations for functions in can.bus

[karlding]

This adds typing annotations for use via mypy for all functions under
can.bus.

This works towards PEP 561 compatibility.

[karlding]

I wasn't sure how to handle annotating stop_all_periodic_tasks, since we wrap the original stop class method and then add support for the remove_tasks argument, so each implementation doesn't need to provide it. Thus, wrapped_stop_method is called first in the call stack before the original stop method.

Adding annotations to stop_all_periodic_tasks would result in that function being checked by mypy, and I wasn't sure how to resolve the following:

can/bus.py:291: error: Unexpected keyword argument "remove_task" for "stop" of "CyclicTask"

As such, I've skipped it for now, but any ideas or insights would be appreciated!

[codecov]

Codecov Report

Merging #652 into develop will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           develop    #652      +/-   ##
==========================================
+ Coverage    67.67%   67.7%   +0.03%     
==========================================
  Files           68      68              
  Lines         6098    6104       +6     
==========================================
+ Hits          4127    4133       +6     
  Misses        1971    1971

[codecov]

Codecov Report

Merging #652 into develop will increase coverage by 0.05%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #652      +/-   ##
===========================================
+ Coverage    67.67%   67.73%   +0.05%     
===========================================
  Files           68       69       +1     
  Lines         6098     6109      +11     
===========================================
+ Hits          4127     4138      +11     
  Misses        1971     1971

[hardbyte]

@karlding given the change in #637 I think we can just remove the remove_task=False from stop_all_periodic_tasks altogether?

[felixdivo]

@karlding given the change in #637 I think we can just remove the remove_task=False from stop_all_periodic_tasks altogether?

But it you then called task.stop() directly, it wouldn't be removed from the bus, right?