Azure / msrest-for-python

The runtime library "msrest" for AutoRest generated Python clients.
MIT License
41 stars 64 forks source link

Add session callback #52

Closed lmazuel closed 7 years ago

lmazuel commented 7 years ago

@jayden-at-arista should fix your scenario.

I don't put a specific config.adapter_kwargs for now, seems to much complicated for a unique scenario.

codecov-io commented 7 years ago

Codecov Report

Merging #52 into master will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #52   +/-   ##
=======================================
  Coverage   77.77%   77.77%           
=======================================
  Files         260      260           
  Lines       12696    12696           
=======================================
  Hits         9874     9874           
  Misses       2822     2822
Impacted Files Coverage Δ
...to_rest_report_service/auto_rest_report_service.py 86.48% <0%> (ø) :arrow_up:
...rmat_test_service/operations/queries_operations.py 76.78% <0%> (ø) :arrow_up:
...rvice/operations/http_server_failure_operations.py 74.28% <0%> (ø) :arrow_up:
...e_test_service/operations/date_model_operations.py 66.43% <0%> (ø) :arrow_up:
...lex/auto_rest_complex_test_service/models/shark.py 100% <0%> (ø) :arrow_up:
...t_service/operations/datetimerfc1123_operations.py 74.39% <0%> (ø) :arrow_up:
...st_url_test_service/operations/paths_operations.py 63.4% <0%> (ø) :arrow_up:
...vice/auto_rest_resource_flattening_test_service.py 69.35% <0%> (ø) :arrow_up:
...onal_test_service/models/class_optional_wrapper.py 80% <0%> (ø) :arrow_up:
...omplex_test_service/operations/basic_operations.py 77.39% <0%> (ø) :arrow_up:
... and 45 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update bad8585...59e052e. Read the comment docs.

jayden-at-arista commented 7 years ago

Changes look good to me. Thanks for following up with this @lmazuel.