Merge pull request reidmorrison#194 from Amnesthesia/fix/rails7.1

fix: remove undef :broadcast since ActiveSupport::Logger dropped it

Author: reidmorrison

.github/workflows/ci.yml

@@ -41,7 +41,15 @@ jobs:
           - rails: "7.0"
             ruby: 3.2
           - rails: "7.0"
-            ruby: jruby 
+            ruby: jruby
+          - rails: "7.1"
+            ruby: "3.0"
+          - rails: "7.1"
+            ruby: 3.1
+          - rails: "7.1"
+            ruby: 3.2
+          - rails: "7.1"
+            ruby: jruby
 
     env:
       BUNDLE_GEMFILE: gemfiles/rails_${{ matrix.rails }}.gemfile

lib/rails_semantic_logger/extensions/active_support/logger.rb

@@ -4,19 +4,22 @@ module ActiveSupport
   # More hacks to try and stop Rails from being it's own worst enemy.
   class Logger
     class << self
-      undef :logger_outputs_to?, :broadcast
+      undef :logger_outputs_to?
+
+      # Prevent broadcasting since SemanticLogger already supports multiple loggers
+      if method_defined?(:broadcast)
+        undef :broadcast
+        def broadcast(logger)
+          Module.new
+        end
+      end
     end
 
     # Prevent Console from trying to merge loggers
     def self.logger_outputs_to?(*args)
       true
     end
 
-    # Prevent broadcasting since SemanticLogger already supports multiple loggers
-    def self.broadcast(logger)
-      Module.new
-    end
-
     def self.new(*args, **kwargs)
       SemanticLogger[self]
     end