chore: Replaced rubocop w/ StandardRb

.github/workflows/ci.yml

@@ -19,8 +19,8 @@ jobs:
           ruby-version: 2.7.8
           bundler-cache: true
 
-      - name: RuboCop
-        run: bundle exec rubocop
+      - name: Standard RB
+        run: bundle exec standardrb
 
   test:
     runs-on: ubuntu-latest

.standard.yml

@@ -0,0 +1,3 @@
+# For available configuration options, see:
+#   https://github.com/standardrb/standard
+ruby_version: 2.7

Gemfile

@@ -1,20 +1,20 @@
 # frozen_string_literal: true
 
-source 'http://rubygems.org'
+source "http://rubygems.org"
 
 gemspec
 
 group :development do
-  gem 'rubocop'
-
-  gem 'delayed_job'
-  gem 'delayed_job_active_record'
-  gem 'rails'
-  gem 'resque'
-  gem 'sidekiq'
-  gem 'sucker_punch'
+  gem "delayed_job"
+  gem "delayed_job_active_record"
+  gem "rails"
+  gem "resque"
+  gem "rubocop"
+  gem "sidekiq"
+  gem "sucker_punch"
+  gem "standard"
 end
 
 group :test do
-  gem 'rspec'
+  gem "rspec"
 end

bin/setup

@@ -0,0 +1,6 @@
+#!/usr/bin/env bash
+set -euo pipefail
+IFS=$'\n\t'
+set -vx
+
+bundle install

lib/generators/vero_generator.rb

@@ -6,10 +6,10 @@ class VeroGenerator < Rails::Generators::Base
   class_option :api_secret
 
   def create_initializer_file
-    type = options[:heroku] || 'standard'
+    type = options[:heroku] || "standard"
 
-    abort('You must provide an API KEY and API SECRET to proceed.') if options[:heroku].blank? && (options[:api_key].blank? || options[:api_secret].blank?)
-    create_file 'config/initializers/vero.rb', send("#{type}_initializer_content")
+    abort("You must provide an API KEY and API SECRET to proceed.") if options[:heroku].blank? && (options[:api_key].blank? || options[:api_secret].blank?)
+    create_file "config/initializers/vero.rb", send("#{type}_initializer_content")
   end
 
   private

lib/vero.rb

@@ -1,55 +1,55 @@
 # frozen_string_literal: true
 
-require 'rest-client'
-require 'vero/utility/ext'
+require "rest-client"
+require "vero/utility/ext"
 
 module Vero
-  autoload :Config,               'vero/config'
-  autoload :App,                  'vero/app'
-  autoload :Context,              'vero/context'
-  autoload :APIContext,           'vero/context/api'
-  autoload :Trackable,            'vero/trackable'
-  autoload :DSL,                  'vero/dsl'
-  autoload :Sender,               'vero/sender'
-  autoload :SuckerPunchWorker,    'vero/senders/sucker_punch'
-  autoload :ResqueWorker,         'vero/senders/resque'
-  autoload :SidekiqWorker,        'vero/senders/sidekiq'
+  autoload :Config, "vero/config"
+  autoload :App, "vero/app"
+  autoload :Context, "vero/context"
+  autoload :APIContext, "vero/context/api"
+  autoload :Trackable, "vero/trackable"
+  autoload :DSL, "vero/dsl"
+  autoload :Sender, "vero/sender"
+  autoload :SuckerPunchWorker, "vero/senders/sucker_punch"
+  autoload :ResqueWorker, "vero/senders/resque"
+  autoload :SidekiqWorker, "vero/senders/sidekiq"
 
   module Api
     module Workers
-      autoload :BaseAPI, 'vero/api/base_api'
+      autoload :BaseAPI, "vero/api/base_api"
 
       module Events
-        autoload :TrackAPI,       'vero/api/events/track_api'
+        autoload :TrackAPI, "vero/api/events/track_api"
       end
 
       module Users
-        autoload :TrackAPI,       'vero/api/users/track_api'
-        autoload :EditAPI,        'vero/api/users/edit_api'
-        autoload :EditTagsAPI,    'vero/api/users/edit_tags_api'
-        autoload :UnsubscribeAPI, 'vero/api/users/unsubscribe_api'
-        autoload :ResubscribeAPI, 'vero/api/users/resubscribe_api'
-        autoload :ReidentifyAPI,  'vero/api/users/reidentify_api'
-        autoload :DeleteAPI,      'vero/api/users/delete_api'
+        autoload :TrackAPI, "vero/api/users/track_api"
+        autoload :EditAPI, "vero/api/users/edit_api"
+        autoload :EditTagsAPI, "vero/api/users/edit_tags_api"
+        autoload :UnsubscribeAPI, "vero/api/users/unsubscribe_api"
+        autoload :ResubscribeAPI, "vero/api/users/resubscribe_api"
+        autoload :ReidentifyAPI, "vero/api/users/reidentify_api"
+        autoload :DeleteAPI, "vero/api/users/delete_api"
       end
     end
 
-    autoload :Events,             'vero/api'
-    autoload :Users,              'vero/api'
+    autoload :Events, "vero/api"
+    autoload :Users, "vero/api"
   end
 
   module Senders
-    autoload :Base,               'vero/senders/base'
-    autoload :DelayedJob,         'vero/senders/delayed_job'
-    autoload :Resque,             'vero/senders/resque'
-    autoload :Sidekiq,            'vero/senders/sidekiq'
-    autoload :Invalid,            'vero/senders/invalid'
-    autoload :SuckerPunch, 'vero/senders/sucker_punch'
+    autoload :Base, "vero/senders/base"
+    autoload :DelayedJob, "vero/senders/delayed_job"
+    autoload :Resque, "vero/senders/resque"
+    autoload :Sidekiq, "vero/senders/sidekiq"
+    autoload :Invalid, "vero/senders/invalid"
+    autoload :SuckerPunch, "vero/senders/sucker_punch"
   end
 
   module Utility
-    autoload :Logger, 'vero/utility/logger'
+    autoload :Logger, "vero/utility/logger"
   end
 end
 
-require 'vero/railtie' if defined?(Rails)
+require "vero/railtie" if defined?(Rails)

lib/vero/api/base_api.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require 'json'
-require 'rest-client'
+require "json"
+require "rest-client"
 
 module Vero
   module Api
@@ -42,16 +42,18 @@ def proxy.<<(message)
           end
         end
 
-        def url; end
+        def url
+        end
 
         def validate!
           raise "#{self.class.name}#validate! should be overridden"
         end
 
-        def request; end
+        def request
+        end
 
         def request_content_type
-          { content_type: :json, accept: :json }
+          {content_type: :json, accept: :json}
         end
 
         def request_params_as_json

lib/vero/api/events/track_api.rb

@@ -14,8 +14,8 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :event_name' if options[:event_name].to_s.blank?
-            raise ArgumentError, ':data must be either nil or a Hash' unless options[:data].nil? || options[:data].is_a?(Hash)
+            raise ArgumentError, "Missing :event_name" if options[:event_name].to_s.blank?
+            raise ArgumentError, ":data must be either nil or a Hash" unless options[:data].nil? || options[:data].is_a?(Hash)
           end
         end
       end

lib/vero/api/users/delete_api.rb

@@ -14,7 +14,7 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id' if options[:id].to_s.blank?
+            raise ArgumentError, "Missing :id" if options[:id].to_s.blank?
           end
         end
       end

lib/vero/api/users/edit_api.rb

@@ -14,8 +14,8 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id or :email' if options[:id].to_s.blank? && options[:email].to_s.blank?
-            raise ArgumentError, ':changes must be a Hash' unless options[:changes].is_a?(Hash)
+            raise ArgumentError, "Missing :id or :email" if options[:id].to_s.blank? && options[:email].to_s.blank?
+            raise ArgumentError, ":changes must be a Hash" unless options[:changes].is_a?(Hash)
           end
         end
       end

lib/vero/api/users/edit_tags_api.rb

@@ -14,10 +14,10 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id or :email' if options[:id].to_s.blank? && options[:email].to_s.blank?
-            raise ArgumentError, ':add must an Array if present' unless options[:add].nil? || options[:add].is_a?(Array)
-            raise ArgumentError, ':remove must an Array if present' unless options[:remove].nil? || options[:remove].is_a?(Array)
-            raise ArgumentError, 'Either :add or :remove must be present' if options[:remove].nil? && options[:add].nil?
+            raise ArgumentError, "Missing :id or :email" if options[:id].to_s.blank? && options[:email].to_s.blank?
+            raise ArgumentError, ":add must an Array if present" unless options[:add].nil? || options[:add].is_a?(Array)
+            raise ArgumentError, ":remove must an Array if present" unless options[:remove].nil? || options[:remove].is_a?(Array)
+            raise ArgumentError, "Either :add or :remove must be present" if options[:remove].nil? && options[:add].nil?
           end
         end
       end

lib/vero/api/users/reidentify_api.rb

@@ -14,8 +14,8 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id' if options[:id].to_s.blank?
-            raise ArgumentError, 'Missing :new_id' if options[:new_id].to_s.blank?
+            raise ArgumentError, "Missing :id" if options[:id].to_s.blank?
+            raise ArgumentError, "Missing :new_id" if options[:new_id].to_s.blank?
           end
         end
       end

lib/vero/api/users/resubscribe_api.rb

@@ -14,7 +14,7 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id or :email' if options[:id].to_s.blank? && options[:email].to_s.blank?
+            raise ArgumentError, "Missing :id or :email" if options[:id].to_s.blank? && options[:email].to_s.blank?
           end
         end
       end

lib/vero/api/users/track_api.rb

@@ -14,8 +14,8 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id or :email' if options[:id].to_s.blank? && options[:email].to_s.blank?
-            raise ArgumentError, ':data must be either nil or a Hash' unless options[:data].nil? || options[:data].is_a?(Hash)
+            raise ArgumentError, "Missing :id or :email" if options[:id].to_s.blank? && options[:email].to_s.blank?
+            raise ArgumentError, ":data must be either nil or a Hash" unless options[:data].nil? || options[:data].is_a?(Hash)
           end
         end
       end

lib/vero/api/users/unsubscribe_api.rb

@@ -14,7 +14,7 @@ def request
           end
 
           def validate!
-            raise ArgumentError, 'Missing :id or :email' if options[:id].to_s.blank? && options[:email].to_s.blank?
+            raise ArgumentError, "Missing :id or :email" if options[:id].to_s.blank? && options[:email].to_s.blank?
           end
         end
       end

lib/vero/app.rb

@@ -9,7 +9,7 @@ def self.default_context
     end
 
     def self.init(&block)
-      default_context.configure(&block) if block_given?
+      default_context.configure(&block) if block
     end
 
     def self.reset!

lib/vero/config.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
-require 'base64'
+require "base64"
 
 module Vero
   class Config
@@ -16,7 +16,7 @@ def initialize
     end
 
     def config_params
-      { tracking_api_key: tracking_api_key }
+      {tracking_api_key: tracking_api_key}
     end
 
     def request_params
@@ -28,9 +28,9 @@ def request_params
 
     def domain
       if @domain.blank?
-        'https://api.getvero.com'
+        "https://api.getvero.com"
       else
-        @domain =~ %r{https?://.+} ? @domain : "http://#{@domain}"
+        %r{https?://.+}.match?(@domain) ? @domain : "http://#{@domain}"
       end
     end
 
@@ -43,10 +43,10 @@ def disable_requests!
     end
 
     def reset!
-      self.disabled         = false
+      self.disabled = false
       self.development_mode = false
-      self.async            = true
-      self.logging          = false
+      self.async = true
+      self.logging = false
       self.tracking_api_key = nil
     end