econchick / econchick/interrogate

-e/--exclude doesn't work with pre-commit

Open
#60 12 comments 0 reactions 1 assignee Claimed by @econchick View on GitHub
bug help wanted
Dominant language
Python
Stars
676
Forks
55
PR merge metrics
No merged PRs in 30d

Description

### Environment

- `interrogate` version: 1.3.1
- Operating System(s): Windows
- Python version(s): 3.7.7

### Description of the bug

Using the `-e/--exclude` argument works perfectly fine when using `interrogate` directly on the command line. However, when I pass it in my `.pre-commit-config.yaml` file like so:

```yaml
- repo: https://github.com/econchick/interrogate
rev: 1.3.1 # or master if you're bold
hooks:
- id: interrogate
args: [--exclude=tests, --fail-under=100]
```
The argument doesn't seem to take effect. So far, I have resulted to using `--ignore-regex` which does work and achieves the same goal. Still, strange that `--exclude` doesn't seem to work. Maybe I am doing something wrong?

### What you expected to happen

I expected `--exclude` to work using pre-commit.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.