Fix ecosystem check for indico

python/ruff-ecosystem/ruff_ecosystem/defaults.py

@@ -3,7 +3,6 @@
 """
 from ruff_ecosystem.projects import (
     CheckOptions,
-    ConfigOverrides,
     FormatOptions,
     Project,
     Repository,
@@ -55,11 +54,6 @@
     Project(repo=Repository(owner="pypa", name="pip", ref="main")),
     Project(
         repo=Repository(owner="pypa", name="setuptools", ref="main"),
-        # Since `setuptools` opts into the "preserve" quote style which
-        # require preview mode, we must disable it during the `--no-preview` run
-        config_overrides=ConfigOverrides(
-            when_no_preview={"format.quote-style": "double"}
-        ),
     ),
     Project(repo=Repository(owner="python", name="mypy", ref="master")),
     Project(
@@ -99,6 +93,66 @@
     ),
     Project(
         repo=Repository(owner="indico", name="indico", ref="master"),
+        # Remove once indico removed S401 from their ignore configuration
+        config_overrides={"lint.ignore": [
+            'E226',  # allow omitting whitespace around arithmetic operators
+            'E731',  # allow assigning lambdas (it's useful for single-line functions defined inside other functions)
+            'N818',  # not all our exceptions are errors
+            'RUF012',  # ultra-noisy and dicts in classvars are very common
+            'RUF015',  # not always more readable, and we don't do it for huge lists
+            'RUF022',  # autofix messes up out formatting instead of just sorting
+            'RUF027',  # also triggers on i18n functions -> too noisy for now
+            'D205',  # too many docstrings which have no summary line
+            'D301',  # https://github.com/astral-sh/ruff/issues/8696
+            'D1',  # we have way too many missing docstrings :(
+            'D401',  # too noisy (but maybe useful to go through at some point)
+            'D412',  # we do not use section, and in click docstrings those blank lines are useful
+            'S101',  # we use asserts outside tests, and do not run python with `-O` (also see B011)
+            'S113',  # enforcing timeouts would likely require config in some places - maybe later
+            'S311',  # false positives, it does not care about the context
+            'S324',  # all our md5/sha1 usages are for non-security purposes
+            'S404',  # useless, triggers on *all* subprocess imports
+            'S403',  # there's already a warning on using pickle, no need to have one for the import
+            'S405',  # we don't use lxml in unsafe ways
+            'S410',  # we don't use lxml in unsafe ways
+            'S603',  # useless, triggers on *all* subprocess calls: https://github.com/astral-sh/ruff/issues/4045
+            'S607',  # we trust the PATH to be sane
+            'B011',  # we don't run python with `-O` (also see S101)
+            'B904',  # possibly useful but too noisy
+            'COM812',  # trailing commas on multiline lists are nice, but we have 2.5k violations
+            'PIE807',  # `lambda: []` is much clearer for `load_default` in schemas
+            'PT004',  # pretty weird + not a pytest convention: https://github.com/astral-sh/ruff/issues/8796
+            'PT005',  # ^ likewise
+            'PT011',  # very noisy
+            'PT015',  # nice for tests but not so nice elsewhere
+            'PT018',  # ^ likewise
+            'SIM102',  # sometimes nested ifs are more readable
+            'SIM103',  # sometimes this is more readable (especially when checking multiple conditions)
+            'SIM105',  # try-except-pass is faster and people are used to it
+            'SIM108',  # noisy ternary
+            'SIM114',  # sometimes separate ifs are more readable (especially if they just return a bool)
+            'SIM117',  # nested context managers may be more readable
+            'PLC0415',  # local imports are there for a reason
+            'PLC2701',  # some private imports are needed
+            'PLR09',  # too-many-<whatever> is just noisy
+            'PLR0913',  # very noisy
+            'PLR2004',  # extremely noisy and generally annoying
+            'PLR6201',  # sets are faster (by a factor of 10!) but it's noisy and we're in nanoseconds territory
+            'PLR6301',  # extremely noisy and generally annoying
+            'PLW0108',  # a lambda often makes it more clear what you actually want
+            'PLW1510',  # we often do not care about the status code of commands
+            'PLW1514',  # we expect UTF8 environments everywhere
+            'PLW1641',  # false positives with SA comparator classes
+            'PLW2901',  # noisy and reassigning to the loop var is usually intentional
+            'TRY002',  # super noisy, and those exceptions are pretty exceptional anyway
+            'TRY003',  # super noisy and also useless w/ werkzeugs http exceptions
+            'TRY300',  # kind of strange in many cases
+            'TRY301',  # sometimes doing that is actually useful
+            'TRY400',  # not all exceptions need exception logging
+            'PERF203',  # noisy, false positives, and not applicable for 3.11+
+            'FURB113',  # less readable
+            'FURB140',  # less readable and actually slower in 3.12+
+        ]},
     ),
     # Jupyter Notebooks
     Project(