Skip to content

Conversation

felixdivo
Copy link
Collaborator

Also cleans up test/config.py a bit.

Related to #951.

@felixdivo felixdivo added this to the 4.0.0 Release milestone Apr 15, 2021
@felixdivo felixdivo self-assigned this Apr 15, 2021
@codecov
Copy link

codecov bot commented Apr 15, 2021

Codecov Report

Merging #1009 (215ab8c) into develop (cd87d2d) will decrease coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop    #1009      +/-   ##
===========================================
- Coverage    70.55%   70.49%   -0.07%     
===========================================
  Files           76       76              
  Lines         7442     7442              
===========================================
- Hits          5251     5246       -5     
- Misses        2191     2196       +5     

@mergify mergify bot requested a review from hardbyte April 15, 2021 08:14
@zariiii9003
Copy link
Collaborator

You could also remove the appveyor references from tox.ini

@felixdivo
Copy link
Collaborator Author

You could also remove the appveyor references from tox.ini

Good catch!

@felixdivo
Copy link
Collaborator Author

After this PR is merged, AppVeyor should probably be unlinked from the project such that it does not show up in the CI section any more.

@hardbyte hardbyte merged commit 40078c7 into develop Apr 15, 2021
@hardbyte hardbyte deleted the remove-app-veyor branch April 15, 2021 21:12
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.

3 participants