progress

Author: psteinroe

crates/pgls_analyser/src/lib.rs

@@ -21,12 +21,11 @@ pub use linter_rule::{LinterDiagnostic, LinterRule};
 
 pub static METADATA: LazyLock<MetadataRegistry> = LazyLock::new(|| {
     let mut metadata = MetadataRegistry::default();
-    // Use a separate visitor for metadata that implements pgls_analyse::RegistryVisitor
     visit_metadata_registry(&mut metadata);
     metadata
 });
 
-// Separate function for visiting metadata registry (uses pgls_analyse::RegistryVisitor)
+// Function for visiting metadata registry (uses pgls_analyse::RegistryVisitor)
 fn visit_metadata_registry<V: pgls_analyse::RegistryVisitor>(registry: &mut V) {
     registry.record_category::<crate::lint::Lint>();
 }

crates/pgls_analyser/src/lint/safety/add_serial_column.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/adding_field_with_default.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/adding_foreign_key_constraint.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/adding_not_null_field.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/adding_primary_key_constraint.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/adding_required_field.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/ban_char_field.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;
@@ -78,7 +78,9 @@ impl LinterRule for BanCharField {
     }
 }
 
-fn check_column_for_char_type(col_def: &pgls_query::protobuf::ColumnDef) -> Option<LinterDiagnostic> {
+fn check_column_for_char_type(
+    col_def: &pgls_query::protobuf::ColumnDef,
+) -> Option<LinterDiagnostic> {
     if let Some(type_name) = &col_def.type_name {
         for name_node in &type_name.names {
             if let Some(pgls_query::NodeEnum::String(name)) = &name_node.node {

crates/pgls_analyser/src/lint/safety/ban_concurrent_index_creation_in_transaction.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;

crates/pgls_analyser/src/lint/safety/ban_drop_column.rs

@@ -1,4 +1,4 @@
-use crate::{LinterRule, LinterRuleContext, LinterDiagnostic};
+use crate::{LinterDiagnostic, LinterRule, LinterRuleContext};
 use pgls_analyse::{RuleSource, declare_lint_rule};
 use pgls_console::markup;
 use pgls_diagnostics::Severity;