MarcoGorelli commented 4 years ago

Hey, I just made a Pull Request!

Description

-e / --exclude doesn't error if the file/directory passed doesn't exist

Motivation and Context

closes #38

Have you tested this? If so, how?

test added

Checklist for PR author(s)

[x] Changes are covered by unit tests (no major decrease in code coverage %).
[x] All tests pass.
[ ] Docstring coverage is 100% via tox -e docs or interrogate -c pyproject.toml (I mean, we should set a good example :smile:).
[ ] Updates to documentation:
- [ ] Document any relevant additions/changes in README.rst.
- [ ] Manually update both the README.rst and docs/index.rst for any new/changed CLI flags.
- [ ] Any changed/added classes/methods/functions have appropriate versionadded, versionchanged, or deprecated directives. Find the appropriate next version in the project's __init__.py file.

Release note

codecov[bot] commented 4 years ago

Codecov Report

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

@@           Coverage Diff           @@
##           master      #39   +/-   ##
=======================================
  Coverage   99.49%   99.49%           
=======================================
  Files           7        7           
  Lines         592      592           
  Branches      125      125           
=======================================
  Hits          589      589           
  Misses          1        1           
  Partials        2        2

Impacted Files	Coverage Δ
src/interrogate/cli.py	`100.00% <ø> (ø)`

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 2105db8...13b38d5. Read the comment docs.

econchick commented 4 years ago

Thank you so much! apologies for the delay in getting to this. I'm cutting a release later today, so this will be in v1.3.0