Verified Commit f4a655e2 authored by Hercules Merscher's avatar Hercules Merscher 🌴
Browse files

fix: Avoiding unnecessary obj instantiation

parent dbaca29f

lib/labkit/covered_experience/experience.rb

+6 −3
Original line number Diff line number Diff line
@@ -160,7 +160,7 @@ module Labkit 
      end



      def build_log_data(event_type, **extra)

        log_data = (extra || {}).merge(

        log_data = {

          checkpoint: event_type,

          covered_experience: @definition.covered_experience,

          feature_category: @definition.feature_category,
@@ -170,13 +170,16 @@ module Labkit 
          end_time: @end_time,

          elapsed_time_s: elapsed_time,

          urgency_threshold_s: urgency_threshold

        ).compact

        }

        log_data.merge!(extra) if extra



        if has_error?

          log_data[:error] = true

          log_data[:error_message] = @error.to_s

          log_data[:error_message] = @error.inspect

        end



        log_data.compact!



        log_data

      end