mistercrunch opened a new pull request, #31447: URL: https://github.com/apache/superset/pull/31447
Ran the `ruff check --fix` but decided against running unsafe fixes. Also ran `ruff check --add-noqa` which is a god send, allowing us to start enforcing new rules, and auto-tagging lines with the right noqa rules. As a follow up, we may want to give `ruff check --usafe-fixes --ignore-noqa` a spin to see try to correct older things marked with noqa as part of this PR, but didn't want to break things as part of this PR Note that I took on a lot more rules here, potentially ones that were not enforced by `pylint` in the past. We can always relax and fine tune rules we don't like moving forward. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
