Skip to content

Commit

Permalink
Update check-manifest invocation
Browse files Browse the repository at this point in the history
check-manifest 0.42 updated `--ignore` handling to be more consistent with MANIFEST.in global-exclude directives.  As a result, `tests*` no longer matches files inside a `tests` directory, and you have to use `tests/**` instead.

As an extra complication, check-manifest drops Python 2.7 support.  Since you still have py27 in this tox.ini, this means you get an older check-manifest 0.41, and you have to tell it to ignore the empty `tests` directory that remains after you exclude all the files in it.

For details see the second half of mgedmin/check-manifest#98

I apologize for the inconvenience!
  • Loading branch information
mgedmin authored May 4, 2020
1 parent d09af3d commit 04b2941
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion tox.ini
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ deps =
flake8
pytest
commands =
check-manifest --ignore tox.ini,tests*
check-manifest --ignore 'tox.ini,tests,tests/**'
# This repository uses a Markdown long_description, so the -r flag to
# `setup.py check` is not needed. If your project contains a README.rst,
# use `python setup.py check -m -r -s` instead.
Expand Down

0 comments on commit 04b2941

Please sign in to comment.