Skip to content

Use EXIT_USAGEERROR instead of magic number#3921

Merged
nicoddemus merged 1 commit intopytest-dev:masterfrom
nicoddemus:use-constant
Sep 1, 2018
Merged

Use EXIT_USAGEERROR instead of magic number#3921
nicoddemus merged 1 commit intopytest-dev:masterfrom
nicoddemus:use-constant

Conversation

@nicoddemus
Copy link
Member

No description provided.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 94.062% when pulling 90c00df on nicoddemus:use-constant into f3b9b21 on pytest-dev:master.

@codecov
Copy link

codecov bot commented Sep 1, 2018

Codecov Report

Merging #3921 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3921      +/-   ##
==========================================
+ Coverage   94.98%   94.98%   +<.01%     
==========================================
  Files         108      108              
  Lines       23360    23361       +1     
  Branches     2324     2324              
==========================================
+ Hits        22188    22189       +1     
  Misses        881      881              
  Partials      291      291
Flag Coverage Δ
#doctesting 28.59% <50%> (+0.07%) ⬆️
#nobyte 89.99% <100%> (ø) ⬆️
#numpy 28.24% <50%> (+0.07%) ⬆️
#pexpect 47.87% <50%> (ø) ⬆️
#pluggymaster 93.72% <100%> (+0.04%) ⬆️
#py27 93.02% <100%> (ø) ⬆️
#py34 92.25% <100%> (+0.03%) ⬆️
#py35 92.27% <100%> (+0.03%) ⬆️
#py36 93.26% <100%> (ø) ⬆️
#py37 92.43% <100%> (+0.03%) ⬆️
#trial 31.19% <50%> (+0.07%) ⬆️
#xdist 93.2% <100%> (ø) ⬆️
Impacted Files Coverage Δ
src/_pytest/config/__init__.py 94.96% <100%> (ø) ⬆️

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 f3b9b21...90c00df. Read the comment docs.

@nicoddemus nicoddemus merged commit dfa7131 into pytest-dev:master Sep 1, 2018
@nicoddemus nicoddemus deleted the use-constant branch September 1, 2018 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants