ops_team 0.11.0.pre → 0.12.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d3a9e4bdba561a28f5b282b09ab717b8445053d68ba715337df0bf7bd294abec
-  data.tar.gz: 7fc1110839291c7912c376597d32a4fcb048952e734dc586c2bf40476a53ec93
+  metadata.gz: 444384ee0931455838357c9bc2f720534e8fbda90d5d27a15c35b313cf676b4e
+  data.tar.gz: 03ff8c264a9083987aac65d9a7ba78fdb8d118e757599c66267725178b8fc809
 SHA512:
-  metadata.gz: 97707fa40c98bae65bc8919da37fd2a366090213fb8fd260339c9489da725779f3d763e06f708d51f70fb03273fa99cb4596f44d663c935515f4b82d3a9d94cf
-  data.tar.gz: 9907f1bcc2b41a4ec0b5e119ffad24c443d76e726b6aca9bec1529d8be6576889a362fd8f56d9cc123fcca8bbe242f30c17b6bf39fb4684e7b745b716532cc28
+  metadata.gz: f596c660d5b7943258c3615a603f36f74ea5d98488d9171562c75593f3e5d5cbe93a349d687a417649dcb0be40de13c9c7d3c33c94b3d7686a43c3f2c94ea2cb
+  data.tar.gz: 486531d3b6c027986cde18d88f3d43e6ffc7db44b4688a8528ccd33b1b0dcb0b883ff6b8048f0b168d3b9fa3d58d8f018722c12a95cdb3edf3e8820c5c3e4b02

data/lib/action.rb

@@ -32,6 +32,10 @@ class Action
 		@config["description"]
 	end
 
+	def skip_hooks?(name)
+		@config["skip_#{name}_hooks"]
+	end
+
 	private
 
 	def load_secrets?

data/lib/builtins/version.rb

@@ -1,13 +1,10 @@
 # frozen_string_literal: true
 
-require "rubygems"
-
 require "output"
+require "version"
 
 module Builtins
 	class Version < Builtin
-		GEMSPEC_FILE = "#{__dir__}/../../ops_team.gemspec"
-
 		class << self
 			def description
 				"prints the version of ops that is running"
@@ -15,18 +12,7 @@ module Builtins
 		end
 
 		def run
-			unless gemspec
-				Output.error("Unable to load gemspec at '#{GEMSPEC_FILE}")
-				return false
-			end
-
-			Output.out(gemspec.version)
-		end
-
-		private
-
-		def gemspec
-			@gemspec ||= Gem::Specification.load(GEMSPEC_FILE)
+			Output.out(::Version.version)
 		end
 	end
 end

data/lib/dependency.rb

@@ -46,7 +46,7 @@ class Dependency
 	end
 
 	def success?
-		@executor&.success?
+		@executor.nil? ? true : @executor.success?
 	end
 
 	def output

data/lib/environment.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'version'
+
 class Environment
 	def initialize(env_hash)
 		@env_hash = env_hash
@@ -21,6 +23,7 @@ class Environment
 
 	def set_ops_variables
 		ENV["OPS_YML_DIR"] = Dir.pwd
+		ENV["OPS_VERSION"] = Version.version.to_s
 	end
 
 	def set_environment_aliases

data/lib/hook_handler.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'output'
+
 class HookHandler
 	class HookConfigError < StandardError; end
 	class HookExecError < StandardError; end
@@ -22,6 +24,7 @@ class HookHandler
 
 	def execute_hooks(name)
 		hooks(name).each do |hook|
+			Output.notice("Running #{name} hook: #{hook}")
 			output, exit_code = Executor.execute(hook)
 
 			next if exit_code.zero?

data/lib/ops.rb

@@ -3,12 +3,14 @@
 
 require 'yaml'
 require 'require_all'
+require "rubygems"
 
 require 'hook_handler'
 require 'action'
 require 'output'
 require 'options'
 require 'environment'
+require 'version'
 require_rel "builtins"
 
 # executes commands based on local `ops.yml`
@@ -20,6 +22,7 @@ class Ops
 	INVALID_SYNTAX_EXIT_CODE = 64
 	UNKNOWN_ACTION_EXIT_CODE = 65
 	ERROR_LOADING_APP_CONFIG_EXIT_CODE = 66
+	MIN_VERSION_NOT_MET_EXIT_CODE = 67
 
 	class << self
 		def project_name
@@ -35,7 +38,9 @@ class Ops
 	end
 
 	def run
+		# "return" is here to allow specs to stub "exit"
 		return exit(INVALID_SYNTAX_EXIT_CODE) unless syntax_valid?
+		return exit(MIN_VERSION_NOT_MET_EXIT_CODE) unless min_version_met?
 
 		run_action
 	rescue UnknownActionError => e
@@ -54,11 +59,28 @@ class Ops
 		end
 	end
 
+	def min_version_met?
+		return true unless min_version
+
+		if Version.min_version_met?(min_version)
+			true
+		else
+			Output.error("ops.yml specifies minimum version of #{min_version}, but ops version is #{Version.version}")
+			false
+		end
+	end
+
+	def min_version
+		config["min_version"]
+	end
+
 	def run_action
-		do_before_run_action
+		do_before_all
 
 		return builtin.run if builtin
 
+		do_before_action
+
 		Output.notice("Running '#{action}' from #{CONFIG_FILE} in environment '#{ENV['environment']}'...")
 		action.run
 	rescue AppConfig::ParsingError => e
@@ -66,10 +88,16 @@ class Ops
 		exit(ERROR_LOADING_APP_CONFIG_EXIT_CODE)
 	end
 
-	def do_before_run_action
+	def do_before_all
 		environment.set_variables
 		AppConfig.load
-		hook_handler.do_hooks("before")
+	end
+
+	def do_before_action
+		hook_handler.do_hooks("before") unless ENV["OPS_RUNNING"] || action.skip_hooks?("before")
+
+		# this prevents before hooks from running in ops executed by ops
+		ENV["OPS_RUNNING"] = "1"
 	end
 
 	def hook_handler

data/lib/version.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require "rubygems"
+
+class Version
+	GEMSPEC_FILE = "#{__dir__}/../ops_team.gemspec"
+
+	class << self
+		# these class methods exist because I don't want to have to call `Version.new.version` elsewhere
+		def version
+			new.send(:version)
+		end
+
+		def min_version_met?(min_version)
+			new.send(:min_version_met?, min_version)
+		end
+	end
+
+	private
+
+	# these instance methods exist so that I don't have to clear class variables between tests
+	def version
+		unless gemspec
+			Output.error("Unable to load gemspec at '#{GEMSPEC_FILE}")
+			return nil
+		end
+
+		gemspec.version
+	end
+
+	def min_version_met?(min_version)
+		Gem::Version.new(version) >= Gem::Version.new(min_version)
+	end
+
+	def gemspec
+		@gemspec ||= Gem::Specification.load(GEMSPEC_FILE)
+	end
+end

data/ops_team.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
 	s.name = 'ops_team'
-	s.version = '0.11.0.pre'
+	s.version = '0.12.3'
 	s.authors = [
 		'nickthecook@gmail.com'
 	]

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ops_team
 version: !ruby/object:Gem::Version
-  version: 0.11.0.pre
+  version: 0.12.3
 platform: ruby
 authors:
 - nickthecook@gmail.com
@@ -176,6 +176,7 @@ files:
 - lib/options.rb
 - lib/output.rb
 - lib/secrets.rb
+- lib/version.rb
 - loader.rb
 - ops_team.gemspec
 homepage: https://github.com/nickthecook/ops
@@ -193,9 +194,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '2.5'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubygems_version: 3.0.3
 signing_key: