Skip to content

Conversation

@tleonhardt
Copy link
Member

@tleonhardt tleonhardt commented Jan 5, 2020

This forces using coverage versions prior to 5.0 since it was a change in this transitive dependency which caused our Windows unit tests to start failing on AppVeyor about 20 days ago.

Closes #836

@tleonhardt tleonhardt added the bug label Jan 5, 2020
@tleonhardt tleonhardt added this to the 1.0.0 milestone Jan 5, 2020
@tleonhardt tleonhardt requested a review from kmvanbrunt January 5, 2020 00:29
@tleonhardt tleonhardt requested a review from kotfu as a code owner January 5, 2020 00:29
@tleonhardt tleonhardt self-assigned this Jan 5, 2020
@codecov
Copy link

codecov bot commented Jan 5, 2020

Codecov Report

Merging #838 into master will increase coverage by 1.12%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #838      +/-   ##
==========================================
+ Coverage   96.23%   97.35%   +1.12%     
==========================================
  Files          14       14              
  Lines        3557     3557              
==========================================
+ Hits         3423     3463      +40     
+ Misses        134       94      -40
Impacted Files Coverage Δ
cmd2/utils.py 97.56% <0%> (+0.6%) ⬆️
cmd2/cmd2.py 96.6% <0%> (+0.93%) ⬆️
cmd2/clipboard.py 100% <0%> (+30.76%) ⬆️
cmd2/rl_utils.py 100% <0%> (+41.46%) ⬆️

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 6a1ec96...54ea40a. Read the comment docs.

@tleonhardt tleonhardt changed the title Try setting a maximum coverage version Set max version restriction for coverage dependency Jan 5, 2020
@tleonhardt tleonhardt merged commit c2ce4b0 into master Jan 6, 2020
@tleonhardt tleonhardt deleted the coverage_version branch January 6, 2020 02:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

AppVeyor tests acting up

3 participants