soy confused

Author: juleswritescode

crates/pglt_analyser/src/lint/safety/ban_drop_table.rs

@@ -18,7 +18,7 @@ declare_lint_rule! {
     pub BanDropTable {
         version: "next",
         name: "banDropTable",
-        recommended: false,
+        recommended: true,
         sources: &[RuleSource::Squawk("ban-drop-table")],
     }
 }

crates/pglt_configuration/src/analyser/linter/rules.rs

@@ -149,18 +149,20 @@ pub struct Safety {
     #[doc = "Dropping a NOT NULL constraint may break existing clients."]
     #[serde(skip_serializing_if = "Option::is_none")]
     pub ban_drop_not_null: Option<RuleConfiguration<pglt_analyser::options::BanDropNotNull>>,
-    #[doc = "Succinct description of the rule."]
+    #[doc = "Dropping a table may break existing clients."]
     #[serde(skip_serializing_if = "Option::is_none")]
     pub ban_drop_table: Option<RuleConfiguration<pglt_analyser::options::BanDropTable>>,
 }
 impl Safety {
     const GROUP_NAME: &'static str = "safety";
     pub(crate) const GROUP_RULES: &'static [&'static str] =
         &["banDropColumn", "banDropNotNull", "banDropTable"];
-    const RECOMMENDED_RULES: &'static [&'static str] = &["banDropColumn", "banDropNotNull"];
+    const RECOMMENDED_RULES: &'static [&'static str] =
+        &["banDropColumn", "banDropNotNull", "banDropTable"];
     const RECOMMENDED_RULES_AS_FILTERS: &'static [RuleFilter<'static>] = &[
         RuleFilter::Rule(Self::GROUP_NAME, Self::GROUP_RULES[0]),
         RuleFilter::Rule(Self::GROUP_NAME, Self::GROUP_RULES[1]),
+        RuleFilter::Rule(Self::GROUP_NAME, Self::GROUP_RULES[2]),
     ];
     const ALL_RULES_AS_FILTERS: &'static [RuleFilter<'static>] = &[
         RuleFilter::Rule(Self::GROUP_NAME, Self::GROUP_RULES[0]),