Lint only project files

[drhagen]

If Python files unrelated to the project get put into the project directory, they get linted and lots of errors appear. For example, setting virtualenvs.in-project = true will cause Poetry to create .venv at the root of the project. This PR changes lint to work like type_check and tells flake8 to only lint the project files. It is unclear if docs should be linted, but it does contain one Python file.

[johnthagen]

I personally prefer keeping these broad as possible to avoid missing having to add a new directory. I'd like to default to having more things checked and exclude them than the other way around. Mypy is an example of something that simply doesn't support this.

I rely on default excludes like black and isort is configured to do to avoid linting the incorrect files, or using exclude.

It appears that flake8 does not exclude venv or .venv by default (unlike black and isort):

• https://github.com/PyCQA/flake8/blob/main/src/flake8/defaults.py#L6

This is one of the reasons I like Poetry's default behaviour to keep the venvs out of the current directory.

I would recommend you also consider excluding .venv, that is what python-blueprint did before the migration to Poetry, but it is a matter of preference which you prefer.

• python-blueprint/tox.ini

  Lines 6 to 7 in d43fb96

  	extend-exclude =
  	venv

[johnthagen]

Another benefit of putting such exclusions into config files rather than only relying on noxfile.py is that CLI usage, or editor integrations will be correct when flake8 is run as well.

[drhagen]

This is actually another benefit of switching to Ruff as suggested in #141. Ruff automatically excludes .venv folders.