Fixes #33682 - Address review comments

adamruzicka · adamruzicka · commit 0873377cd8d3 · 2021-10-12T15:31:44.000+02:00
diff --git a/lib/smart_proxy_remote_execution_ssh.rb b/lib/smart_proxy_remote_execution_ssh.rb
@@ -2,7 +2,6 @@
 require 'smart_proxy_remote_execution_ssh/version'
 require 'smart_proxy_remote_execution_ssh/plugin'
 require 'smart_proxy_remote_execution_ssh/webrick_ext'
-require 'sequel'
 
 module Proxy::RemoteExecution
   module Ssh
diff --git a/lib/smart_proxy_remote_execution_ssh/job_storage.rb b/lib/smart_proxy_remote_execution_ssh/job_storage.rb
@@ -1,4 +1,5 @@
 # lib/job_storage.rb
+require 'sequel'
 
 module Proxy::RemoteExecution::Ssh
   class JobStorage
diff --git a/lib/smart_proxy_remote_execution_ssh/plugin.rb b/lib/smart_proxy_remote_execution_ssh/plugin.rb
@@ -1,7 +1,7 @@
 module Proxy::RemoteExecution::Ssh
   class Plugin < Proxy::Plugin
     SSH_LOG_LEVELS = %w[debug info warn error fatal].freeze
-    MODES = %i[ssh async-ssh pull pull-mqtt]
+    MODES = %i[ssh async-ssh pull pull-mqtt].freeze
 
     http_rackup_path File.expand_path("http_config.ru", File.expand_path("../", __FILE__))
     https_rackup_path File.expand_path("http_config.ru", File.expand_path("../", __FILE__))

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,5 @@`
`1`	`1`	`# lib/job_storage.rb`
	`2`	`+require 'sequel'`
`2`	`3`
`3`	`4`	`module Proxy::RemoteExecution::Ssh`
`4`	`5`	`class JobStorage`