Skip to content

Conversation

angerrp
Copy link
Contributor

@angerrp angerrp commented Dec 28, 2019

Added a simple exclude command line option which can be used to exclude folders and files. Further site-packages get excluded.

Exclusion can be triggered via the -e or --exclude flag following multiple folder or file names e.g. -e <folder> <folder> <file>. Files and folders are excluded in the find_py_files method via re.search.

@coveralls
Copy link

coveralls commented Dec 28, 2019

Coverage Status

Coverage decreased (-0.3%) to 99.307% when pulling 0a43cac on etimoz:add-exclude-option into 346833b on myint:master.

docformatter.py Outdated
@@ -38,6 +38,7 @@
import sys
import textwrap
import tokenize
from distutils.sysconfig import get_python_lib
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you try with standard sysconfig module instead of distutils?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure, can do. Thx for the input

Copy link
Member

@myint myint left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@myint myint merged commit 020ec4f into PyCQA:master May 17, 2020
@enkidulan enkidulan mentioned this pull request Jun 29, 2020
@weibullguy weibullguy added P: enhancement Feature that is outside the scope of PEP 257 C: stakeholder Relates to docformatter stakeholder requested behavior labels Aug 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: stakeholder Relates to docformatter stakeholder requested behavior P: enhancement Feature that is outside the scope of PEP 257
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants