Skip to content
This repository has been archived by the owner on Mar 24, 2021. It is now read-only.

upgrade pytest dependency #839

Merged
merged 8 commits into from
Jul 24, 2018
Merged

upgrade pytest dependency #839

merged 8 commits into from
Jul 24, 2018

Conversation

emmettbutler
Copy link
Contributor

This pull request fixes #782 by upgrading the pytest dependency and removing the code used to work around pytest-dev/pytest#568.

@codecov-io
Copy link

codecov-io commented Jul 23, 2018

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #839   +/-   ##
=======================================
  Coverage   83.98%   83.98%           
=======================================
  Files          27       27           
  Lines        3621     3621           
  Branches      547      547           
=======================================
  Hits         3041     3041           
+ Misses        437      435    -2     
- Partials      143      145    +2
Impacted Files Coverage Δ
pykafka/partition.py 72.72% <0%> (-12.13%) ⬇️
pykafka/topic.py 82.47% <0%> (-2.07%) ⬇️
pykafka/simpleconsumer.py 85.01% <0%> (-0.9%) ⬇️
pykafka/cluster.py 73.03% <0%> (-0.75%) ⬇️
pykafka/handlers.py 93.38% <0%> (ø) ⬆️
pykafka/protocol.py 94.11% <0%> (+0.21%) ⬆️
pykafka/balancedconsumer.py 89.47% <0%> (+0.29%) ⬆️
pykafka/broker.py 92.98% <0%> (+3.5%) ⬆️
pykafka/managedbalancedconsumer.py 84.1% <0%> (+4.63%) ⬆️

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 0f9a0f8...f771c04. Read the comment docs.

@emmettbutler emmettbutler merged commit 23a898b into master Jul 24, 2018
@emmettbutler emmettbutler deleted the bugfix/pytest_skip branch July 24, 2018 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

simplify test skipping logic
2 participants