#88: change up inclusions, exclusions, and best practices from PR fee…

…dback.

pyproject.toml

@@ -120,7 +120,6 @@ select = [
     "FBT",  # flake8-boolean-trap
     "FIX",  # flake8-fixme
     "FLY",  # flynt
-    "FURB",  # refurb
     "I",  # isort
     "ICN",  # flake8-import-conventions
     "ISC",  # flake8-implicit-str-concat
@@ -141,32 +140,30 @@ select = [
     "UP",  # python upgrade
     "W",  # pycodestyle warning
     "YTT",  # flake8-2020
+
+    # we would like these someday, but not yet
+    # "FURB",  # refurb
 ]
 ignore = [
     "D107",  # missing __init__ docstring, we do that in the class docstring.
-    "D203",  # one blank line before class; we want 2!
+    "D203",  # one blank line before class docstring, no thanks!
     "D212",  # multi line summary first line, we want a one line summary.
     "ANN101",  # missing self annotation, we only annotate self when we return it.
     "ANN401",  # no `**kwargs: Any` typing; unfortunately we don't have a better way.
 ]
-include = [
-    "screenpy",
-    "tests",
-]
 exclude = [
     "screenpy/__init__.py",
     "screenpy/__version__.py",
+    "docs",
 ]
 
 
 [tool.ruff.per-file-ignores]
-"__init__.py" = ["F401"]
-"tests/ui/admin/budget_configure/modal*.py" = ["F401"]
-"tests/ui/admin/organization_configure/modal*.py" = ["F401"]
-"tests/ui/admin/schedule_configure/modal*.py" = ["F401"]
-"setup/dbutil.py" = ["E722"] # bare except
-"tests/question/new_expected_conditions.py" = ["ANN"] # this is a copy of the original
-
+"tests/**" = [
+    "D",  # we don't need public-API-polished docstrings in tests.
+    "FBT",  # using a boolean as a test object is useful!
+    "PLR",  # likewise using specific numbers and strings in tests.
+]
 
 [tool.ruff.isort]
 combine-as-imports = true