kpn / py-timeexecution

Time Execution: record application metrics
https://pypi.org/project/timeexecution/
Apache License 2.0
12 stars 19 forks source link

BREAK: drop old python versions, refactor CI #49

Closed aikikode closed 4 years ago

aikikode commented 4 years ago

NB: do not merge until https://github.com/kpn-digital/py-fqn-decorators/pull/19 is merged and we have tag 2.0.0 for fqn-decorators to check py-timeexecution works with version >=2.0

Changes here:

codecov-io commented 4 years ago

Codecov Report

Merging #49 (94d50d2) into master (2bb572c) will decrease coverage by 1.51%. The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #49      +/-   ##
==========================================
- Coverage   95.63%   94.11%   -1.52%     
==========================================
  Files           9        8       -1     
  Lines         275      272       -3     
  Branches        0       26      +26     
==========================================
- Hits          263      256       -7     
  Misses         12       12              
- Partials        0        4       +4     
Impacted Files Coverage Δ
time_execution/backends/base.py 100.00% <100.00%> (ø)
time_execution/backends/elasticsearch.py 97.77% <100.00%> (-2.23%) :arrow_down:
time_execution/backends/influxdb.py 100.00% <100.00%> (ø)
time_execution/backends/kafka.py 97.56% <100.00%> (-2.44%) :arrow_down:
time_execution/backends/threaded.py 100.00% <100.00%> (ø)
time_execution/decorator.py 100.00% <100.00%> (ø)
time_execution/serializer.py 50.00% <100.00%> (-7.15%) :arrow_down:

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 2bb572c...94d50d2. Read the comment docs.

kammala commented 4 years ago

LGTM, but coverage has decreased for some reason.