Merge pull request #5 from snacks02/ameba

Add Ameba for linting

.ameba.yml

@@ -0,0 +1,5 @@
+Documentation/DocumentationAdmonition:
+  Enabled: false
+
+Lint/NotNil:
+  Enabled: false

.github/workflows/crystal.yml

@@ -56,6 +56,8 @@ jobs:
         run: shards install
       - name: Check code style
         run: crystal tool format --check
+      - name: Check code linting with Ameba
+        run: bin/ameba
       - name: Run tests
         env:
           DATABASE_URL: postgres://example:example@postgres:5432/example

shard.yml

@@ -12,6 +12,8 @@ dependencies:
   tallboy:
     github: epoch/tallboy
 development_dependencies:
+  ameba:
+    github: crystal-ameba/ameba
   sqlite3:
     github: crystal-lang/crystal-sqlite3
   mysql:

spec/table_spec.cr

@@ -7,7 +7,7 @@ describe Cql::Table do
 
     customer = CustomerModel.new(1, "John", "New York", 100)
 
-    insert_query = TableDB.insert.into(:customers).values(
+    TableDB.insert.into(:customers).values(
       id: customer.id,
       name: customer.name,
       city: customer.city,
@@ -40,7 +40,7 @@ describe Cql::Table do
     count_query = TableDB.query.from(:customers).count
     count_query.first!(as: Int32).should eq 2
 
-    result = TableDB.delete.from(:customers).commit
+    TableDB.delete.from(:customers).commit
     count_query.first!(as: Int32).should eq 0
   end
 
@@ -50,7 +50,7 @@ describe Cql::Table do
     check_query = "SELECT name FROM sqlite_master WHERE type='table' AND name='#{table}'"
 
     expect_raises(DB::NoResultsError) do
-      name = TableDB.db.query_one(check_query, as: String)
+      TableDB.db.query_one(check_query, as: String)
     end
   end
 end

src/alter_table.cr

@@ -199,15 +199,15 @@ module Cql
     # sql = to_sql(visitor)
     # ```
     def to_sql(visitor : Expression::Visitor)
-      String::Builder.build do |sb|
+      String.build do |string|
         @actions.each do |action|
           case action
           when Expression::CreateIndex, Expression::DropIndex, Expression::RenameTable
-            sb << action.accept(visitor)
+            string << action.accept(visitor)
           else
-            sb << Expression::AlterTable.new(@table, action).accept(visitor)
+            string << Expression::AlterTable.new(@table, action).accept(visitor)
           end
-          sb << ";\n"
+          string << ";\n"
         end
       end
     end

src/dialects/mysql_dialect.cr

@@ -10,12 +10,12 @@ module Expression
     end
 
     def auto_increment_primary_key(column : Cql::BaseColumn, col_type : String) : String
-      String::Builder.build do |sb|
-        sb << column.name
-        sb << " "
-        sb << col_type
-        sb << " PRIMARY KEY"
-        sb << " AUTO_INCREMENT" if column.auto_increment?
+      String.build do |string|
+        string << column.name
+        string << " "
+        string << col_type
+        string << " PRIMARY KEY"
+        string << " AUTO_INCREMENT" if column.auto_increment?
       end
     end
 

src/dialects/postgres_dialect.cr

@@ -11,13 +11,13 @@ module Expression
     end
 
     def auto_increment_primary_key(column : Cql::BaseColumn, col_type : String) : String
-      String::Builder.build do |sb|
-        sb << column.name
-        sb << " "
-        sb << col_type
-        sb << " GENERATED"
-        sb << " ALWAYS" if column.auto_increment?
-        sb << " AS IDENTITY PRIMARY KEY"
+      String.build do |string|
+        string << column.name
+        string << " "
+        string << col_type
+        string << " GENERATED"
+        string << " ALWAYS" if column.auto_increment?
+        string << " AS IDENTITY PRIMARY KEY"
       end
     end
 

src/dialects/sqlite_dialect.cr

@@ -33,7 +33,7 @@ module Expression
     end
 
     def drop_foreign_key(table_name : String, constraint_name : String) : String
-      message = <<-MSG
+      <<-MSG
       SQLite does not support dropping foreign keys directly via the ALTER TABLE
       statement. You need to recreate the table without the foreign key constraint.
 

src/expression/aggregator_builder.cr

@@ -5,28 +5,28 @@ module Expression
     def initialize(@aggregate_function : Condition)
     end
 
-    def >(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, ">", value))
+    def >(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, ">", other))
     end
 
-    def <(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "<", value))
+    def <(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "<", other))
     end
 
-    def >=(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, ">=", value))
+    def >=(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, ">=", other))
     end
 
-    def <=(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "<=", value))
+    def <=(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "<=", other))
     end
 
-    def ==(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "=", value))
+    def ==(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "=", other))
     end
 
-    def !=(value : Column | DB::Any)
-      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "!=", value))
+    def !=(other : Column | DB::Any)
+      ConditionBuilder.new(CompareCondition.new(@aggregate_function, "!=", other))
     end
   end
 end