New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
References rule bundle #4446
References rule bundle #4446
Conversation
Pull Request Test Coverage Report for Build 4307518570
💛 - Coveralls |
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #4446 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 210 212 +2
Lines 15473 15497 +24
=========================================
+ Hits 15473 15497 +24
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🌈 ✨ Nicely done
@@ -281,7 +281,7 @@ require_final_semicolon = False | |||
# GROUP BY/ORDER BY column references | |||
group_by_and_order_by_style = consistent | |||
|
|||
[sqlfluff:rules:L057] | |||
[sqlfluff:rules:RF05] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are there any plans to allow using the new "readable" rule names in the config rather than the codes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll slack you about this. Short answer - not at the moment because if we accept more than one way of specifying config, we then have to deal with resolving priority.
Another step in #4031. These ones are all straightforward recodes. All the rules are still intact and just moved into new bundles as per the rules spreadsheet.
The only deeper functionality change here is the RF04, RF05 and CP02 shared a utility function called
identifiers_policy_applicable
which had been living in thecapitalisation
bundle. With the new plugin structure, having bundles which depend on other bundles doesn't work neatly (and results in circular imports). To resolve that I've mode this method into thesqlfluff.util
module so that both can depend on it neatly.