search

mohkale / flymake-collection

Collection of checkers for flymake
MIT License
68 stars 13 forks source link

Feature Request: Allow ruff to read from config #28

Open jgarte opened 1 year ago

jgarte commented 1 year ago

Hi,

ruff can read from config files in the repo:

https://github.com/erickgnavar/flymake-ruff/blob/master/flymake-ruff.el#L38

WDYT

mohkale commented 1 year ago

Happy to accept a PR for this, but I'm really not a big fan of linters that expect you to figure out the project type and pass through this sort of parent configuration file they may need. Really ruff should just take the path to the file you're wanting to format as an extra option it can figure if there's any configs or something else in the project that it may want to use for formatting.

jgarte commented 1 year ago

Ruff also has this command line flag: --config

https://docs.astral.sh/ruff/configuration/#command-line-interface