lvgig / tubular

Python package implementing transformers for pre processing steps for machine learning.
https://tubular.readthedocs.io/en/latest/index.html
BSD 3-Clause "New" or "Revised" License
37 stars 14 forks source link

Resolve conflict in ruff rules #260

Open davidhopkinson26 opened 1 week ago

davidhopkinson26 commented 1 week ago

What?

Removing ruff rules which conflict with ruff format

Why?

Currently getting this warning from ruff format:

""" warning: The following rules may cause conflicts when used with the formatter: COM812, ISC001. To avoid unexpected behavior, we recommend disabling these rules, either by removing them from the select or extend-select configuration, or adding them to the ignore configuration. """

missing-trailing-comma (COM812)

single-line-implicit-string-concatenation (ISC001)

How?

Neither of these seem particularly important so I propose we add these rules to lint.ignore in ruff.toml as suggested to avoid future headaches.