Ensure $? is not nil

themucha · Mar 16, 2015 · e8f573d · e8f573d
1 parent 59b4da7
commit e8f573d
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Ruby/lib/terminal-notifier.rb b/Ruby/lib/terminal-notifier.rb
@@ -50,7 +50,7 @@ def self.execute(verbose, options)
   # Raises if not supported on the current platform.
   def notify(message, options = {}, verbose = false)
     TerminalNotifier.execute(verbose, options.merge(:message => message))
-    $?.success?
+    $? && $?.success?
   end
   module_function :notify
 
@@ -60,7 +60,7 @@ def notify(message, options = {}, verbose = false)
   # If no ‘group’ ID is given, all notifications are removed.
   def remove(group = 'ALL', verbose = false)
     TerminalNotifier.execute(verbose, :remove => group)
-    $?.success?
+    $? && $?.success?
   end
   module_function :remove