more build improvements

.github/workflows/push.yml

@@ -53,3 +53,17 @@ jobs:
           PGUSER: postgres
           PGPASSWORD: postgres
           BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+  lint:
+    runs-on: ubuntu-latest
+
+    steps:
+      - uses: actions/checkout@v2
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.7
+          bundler-cache: true
+      - name: Install dependencies
+        run: bundle install
+      - name: Run rubocop
+        run: bundle exec rake rubocop

.rubocop.yml

@@ -3,6 +3,7 @@ AllCops:
   NewCops: enable
   Exclude:
     - 'gemfiles/*'
+    - 'vendor/bundle/**/*'
 
 Style/StringLiterals:
   Enabled: true

Gemfile.lock

@@ -51,6 +51,8 @@ GEM
     minitest (5.15.0)
     nokogiri (1.13.3-x86_64-darwin)
       racc (~> 1.4)
+    nokogiri (1.13.3-x86_64-linux)
+      racc (~> 1.4)
     parallel (1.20.1)
     parser (3.0.1.1)
       ast (~> 2.4.1)

Rakefile

@@ -9,4 +9,4 @@ require "rubocop/rake_task"
 
 RuboCop::RakeTask.new
 
-task default: %i[spec rubocop]
+task default: %i[spec]

spec/spec_helper.rb

@@ -29,7 +29,7 @@ def executed_statements
 
   %w[execute exec_no_cache exec_cache].each do |method|
     class_eval <<-RUBY, __FILE__, __LINE__ + 1
-      def #{method}(statement, *, **kwargs)
+      def #{method}(statement, *#{Rails.version >= '7.0' ? ', **kwargs' : ''})
         materialize_transactions # this still needs to get called, even if we skip actually executing
         executed_statements << statement
         return empty_pg_result if @dont_execute