René's URL Explorer Experiment

Title: Respect Ruff rule code hierarchy in select/ignore lists by Copilot · Pull Request #1527 · usethis-python/usethis-python · GitHub

Open Graph Title: Respect Ruff rule code hierarchy in select/ignore lists by Copilot · Pull Request #1527 · usethis-python/usethis-python

X Title: Respect Ruff rule code hierarchy in select/ignore lists by Copilot · Pull Request #1527 · usethis-python/usethis-python

Description: Add is_rule_covered_by() and reconcile_rules() helpers in _tool/rule.py Introduce RuleReconciliation frozen dataclass (replaces tuple return) Update select_rules / ignore_rules in _tool/base.py to use reconciliation Update ignore_rules_in_glob in _tool/impl/base/ruff.py to use reconciliation Fix bug: startswith was too crude — now requires a digit boundary after prefix match (e.g. F covers F101 but not FLY) Add tests for cross-group scenarios (FLY vs F, FLY001 vs F, F101 vs F1) Run full static checks and verify Original prompt This section details on the original issue you should resolve Respect Ruff rule code hierarchy If Ruff selection contains "ALL" then we're already selecting all rules! There's not need to explicitly select another rule, it just gets "absorbed" into this "wildcard" style rule. Similarly, if we want to select a specific rule code like TC001 then if there's already the rule code group like TC then there's no point adding it - we shouldn't. It also goes the other way. We should replace a less general rule with a more general one. If we want to select TC and there's already TC001 then we should replace TC001 with TC. All these principles also apply to ignores! And per-file ignores! Any time there's a list of ruff rules. We should probably keep the current level of abstraction which manages the raw rule lists, but basically move that into helper methods, and so we can add some smart logic on top that manages the hierarchy between rule codes. It is safe to assume that all rule codes follow this pattern - not just Ruff. Comments on the Issue (you are @copilot in this section) Fixes #1526 ✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Open Graph Description: Add is_rule_covered_by() and reconcile_rules() helpers in _tool/rule.py Introduce RuleReconciliation frozen dataclass (replaces tuple return) Update select_rules / ignore_rules in _tool/base.py t...

X Description: Add is_rule_covered_by() and reconcile_rules() helpers in _tool/rule.py Introduce RuleReconciliation frozen dataclass (replaces tuple return) Update select_rules / ignore_rules in _tool/base.py t...

Opengraph URL: https://github.com/usethis-python/usethis-python/pull/1527

X: @github

direct link

Domain: github.com

Links:

Viewport: width=device-width