Merge pull request #18340 from Homebrew/revert-18329-limit-annotations

Revert "Limit usage of GitHub Actions Annotations"
Homebrew · Sep 16, 2024 · 08e5122 · 08e5122
2 parents 547677c + 1c39009
commit 08e5122
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 11 deletions.
diff --git a/Library/Homebrew/extend/kernel.rb b/Library/Homebrew/extend/kernel.rb
@@ -65,11 +65,8 @@ def oh1(title, truncate: :auto)
   sig { params(message: T.any(String, Exception)).void }
   def opoo(message)
     Tty.with($stderr) do |stderr|
-      if ENV["HOMEBREW_GITHUB_ACTIONS"].present?
-        GitHub::Actions.puts_annotation_if_env_set(:warning, message.to_s)
-      else
-        stderr.puts Formatter.warning(message, label: "Warning")
-      end
+      stderr.puts Formatter.warning(message, label: "Warning")
+      GitHub::Actions.puts_annotation_if_env_set(:warning, message.to_s)
     end
   end
 
@@ -82,11 +79,8 @@ def onoe(message)
     require "utils/github/actions"
 
     Tty.with($stderr) do |stderr|
-      if ENV["HOMEBREW_GITHUB_ACTIONS"].present?
-        GitHub::Actions.puts_annotation_if_env_set(:error, message.to_s)
-      else
-        stderr.puts Formatter.error(message, label: "Error")
-      end
+      stderr.puts Formatter.error(message, label: "Error")
+      GitHub::Actions.puts_annotation_if_env_set(:error, message.to_s)
     end
   end
 

diff --git a/Library/Homebrew/utils/github/actions.rb b/Library/Homebrew/utils/github/actions.rb
@@ -51,7 +51,7 @@ def self.puts_annotation_if_env_set(type, message, file: nil, line: nil)
       return unless env_set?
 
       std = (type == :notice) ? $stdout : $stderr
-      std.puts Annotation.new(type, message, file:, line:)
+      std.puts Annotation.new(type, message)
     end
 
     # Helper class for formatting annotations on GitHub Actions.