Minor update of ruff rules, removed init in src

Author: MischaPanch

{{cookiecutter.project_name}}/pyproject.toml

@@ -73,54 +73,134 @@ exclude = "^build/|^docs/"
 [tool.doc8]
 max-line-length = 1000
 
-[tool.black]
-line-length = 100
-target-version = ["py311"]
-
 [tool.nbqa.exclude]
 ruff = "\\.jupyter_cache|jupyter_execute"
 mypy = "\\.jupyter_cache|jupyter_execute"
 
+
+[tool.black]
+line-length = 140
+target-version = ["py311"]
+
 [tool.ruff]
+target-version = "py311"
+line-length = 140
+
+[tool.ruff.lint]
 select = [
-    "ASYNC", "B", "C4", "C90", "COM", "D", "DTZ", "E", "F", "FLY", "G", "I", "ISC", "PIE", "PLC", "PLE", "PLW", "RET", "RUF", "RSE", "SIM", "TID", "UP", "W", "YTT",
+	"ASYNC",
+	"B",
+	"C4",
+	"C90",
+	"COM",
+	"D",
+	"DTZ",
+	"E",
+	"F",
+	"FLY",
+	"G",
+	"I",
+	"ISC",
+	"PIE",
+	"PLC",
+	"PLE",
+	"PLW",
+	"RET",
+	"RUF",
+	"RSE",
+	"SIM",
+	"TID",
+	"UP",
+	"W",
+	"YTT",
 ]
 ignore = [
-    "COM812",  # requires trailing commas and can cause undesirable reformats when calling `poe format` multiple times (see https://github.com/appliedAI-Initiative/pymetrius/issues/17)
-    "E501", # line too long. black does a good enough job
-    "E741", # variable names like "l". this isn't a huge problem
-    "B008", # do not perform function calls in argument defaults. we do this sometimes
-    "B011", # assert false. we don't use python -O
-    "B028", # we don't need explicit stacklevel for warnings
-    "D100", "D101", "D102", "D104", "D105", "D107", "D203", "D213", "D401", "D402", # docstring stuff
-    "DTZ005", # we don't need that
-    # remaining rules from https://github.com/psf/black/blob/main/.flake8 (except W503)
-    # this is a simplified version of config, making vscode plugin happy
-    "E402", "E501", "E701", "E731", "C408", "E203",
-    # Logging statement uses f-string warning
-    "G004",
-    # Unnecessary `elif` after `return` statement
-    "RET505",
-    "D106", # undocumented public nested class
-    "D205", # blank line after summary (prevents summary-only docstrings, which makes no sense)
+	"RUF002", # hyphen checks
+	"RUF005", # unpacking over concatenation (no point in that rule IMO)
+	"SIM118", # Needed b/c iter(batch) != iter(batch.keys()). See https://github.com/thu-ml/tianshou/issues/922
+	"SIM108", # if - else is fine
+	"E501",   # line too long. black does a good enough job
+	"E741",   # variable names like "l". this isn't a huge problem
+	"B008",   # do not perform function calls in argument defaults. we do this sometimes
+	"B011",   # assert false. we don't use python -O
+	"B028",   # we don't need explicit stacklevel for warnings
+	"D100",
+	"D101",
+	"D102",
+	"D103",
+	"D104",
+	"D105",
+	"D107",
+	"D200",
+	"D203",
+	"D213",
+	"D401",
+	"D402",   # docstring stuff
+	"DTZ005", # we don't need that
+	# remaining rules from https://github.com/psf/black/blob/main/.flake8 (except W503)
+	# this is a simplified version of config, making vscode plugin happy
+	"E402",
+	"E501",
+	"E701",
+	"E731",
+	"C408",
+	"E203",
+	# Logging statement uses f-string warning
+	"G004",
+	# Unnecessary `elif` after `return` statement
+	"RET505",
+	"D106",    # undocumented public nested class
+	"D205",    # blank line after summary (prevents summary-only docstrings, which makes no sense)
+	"D212",    # no blank line after """. This clashes with sphinx for multiline descriptions of :param: that start directly after """
+	"PLW2901", # overwrite vars in loop
+	"B027",    # empty and non-abstract method in abstract class
+	"D404",    # It's fine to start with "This" in docstrings
+	"D407",
+	"D408",
+	"D409",    # Ruff rules for underlines under 'Example:' and so clash with Sphinx
+	"D400",    # Period at the end of docstring
+	"D415",    # Closing punctuation
+	"COM812",  # missing trailing comma: With this enabled, re-application of "poe format" chain can cause additional commas and subsequent reformatting
+	"RET503",  # no need to return None
+	"RET504",  # it is fine to return after creation
+	"UP038",   # tuples in isinstance
+	"F403", "F405",    # star imports
+	"C401",    # generators
+	"C901",    # complexity analysis
+
 ]
 unfixable = [
-
+	"F841", # unused variable. ruff keeps the call, but mostly we want to get rid of it all
+	"F601", # automatic fix might obscure issue
+	"F602", # automatic fix might obscure issue
+	"B018", # automatic fix might obscure issue
 ]
 extend-fixable = [
-    "F401", # unused import
-    "B905", # bugbear
+	"F401", # unused import
+	"B905", # bugbear
+	"W291", # trailing whitespace
 ]
 
-target-version = "py311"
+
+[tool.ruff.lint.per-file-ignores]
+"tests/**" = ["D103"]
+"docs/**" = ["D103"]
+"examples/**" = ["D103"]
+"scripts/**" = ["D103"]
+
+
+[tool.ruff.format]
+# configuration to match the Black settings
+quote-style = "double"
+indent-style = "space"
+line-ending = "auto"
+skip-magic-trailing-comma = false
+docstring-code-format = true
+
 
 [tool.ruff.mccabe]
 max-complexity = 20
 
-[tool.ruff.per-file-ignores]
-"test/**" = ["D103"]
-"docs/**" = ["D103"]
-"examples/**" = ["D103"]
 
 [tool.poetry_bumpversion.file."{{cookiecutter.package_name}}/__init__.py"]
 
@@ -132,8 +212,6 @@ PYDEVD_DISABLE_FILE_VALIDATION="1"
 # keep relevant parts in sync with pre-commit
 [tool.poe.tasks]  # https://github.com/nat-n/poethepoet
 test = "pytest test --cov={{cookiecutter.package_name}} --cov-report=xml --cov-report=term-missing --durations=0 -v --color=yes"
-# Adjust to a smaller set of tests if appropriate
-test-subset = "pytest test --color=yes"
 _black_check = "black --check ."
 _ruff_check = "ruff check ."
 _ruff_check_nb = "nbqa ruff docs"