mistercrunch opened a new pull request, #35081:
URL: https://github.com/apache/superset/pull/35081
- Enable G-group logging rules (G001-G999) for better logging practices
- Add UP, RUF, FBT, SIM rule groups for code quality and modernization
- Fix 190+ G-rule violations: f-strings and exc_info logging patterns
- Apply conservative fixes across 600+ files:
* Boolean API improvements (FBT rules) for clearer function signatures
* Code simplification (SIM rules) for better readability
* Modern Python patterns (UP rules) for type annotations
* Ruff-specific improvements (RUF rules) for code quality
- Add strategic noqa comments for legitimate rule exceptions
- Disable RUF012 (mutable class attributes) - often intentional configuration
- Temporarily disable E501 (line-too-long) after auto-fix removed noqa
comments
- Exclude databases/utils.py from blacklist hook (legitimate make_url_safe
wrapper)
--
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: notifications-unsubscr...@superset.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org
For additional commands, e-mail: notifications-h...@superset.apache.org
