logger-f v2.0.0-beta22

[kevin-lee]

logger-f v2.0.0-beta22

2.0.0-beta22 - 2023-11-08

Change

• Change the LogMessage parameter in the log(F[A]) method from NotIgnorable to MaybeIgnorable (Change the LogMessage parameter in the log(F[A]) method from NotIgnorable to MaybeIgnorable #498)
  This could be required and useful for case like

  final case class Something(id: Int, name: String)
  
  val fa: F[Something] = ...
  Log[F].log(fa) {
    case Something(0, _) => ignore
    case Something(n, name) => info(s"Something: id=$n, name=$name")
  }

Improvement

• Remove unnecessary re-evaluation of String in logS (Remove unnecessary re-evaluation of String in logS #500)
  msg() and message (call-by-name) below in line 61 (Scala 2) and line 62 (Scala 3) were replaced with a single lazy evaluation.

  logger-f/modules/logger-f-core/shared/src/main/scala-2/loggerf/core/Log.scala

  Lines 56 to 62 in 47a0ad1

  	def logS(
  	message: => String
  	)(toLeveledMessage: (String => LogMessage with NotIgnorable) with LogMessage.LeveledMessage.Leveled): F[String] =
  	toLeveledMessage.toLazyInput(message) match {
  	case LogMessage.LeveledMessage(msg, level) =>
  	map0(EF.effectOf(canLog.getLogger(level)(msg())))(_ => message)
  	}

  logger-f/modules/logger-f-core/shared/src/main/scala-3/loggerf/core/Log.scala

  Lines 59 to 63 in 47a0ad1

  	def logS(message: => String)(toLeveledMessage: (String => LeveledMessage) with LeveledMessage.Leveled): F[String] =
  	toLeveledMessage.toLazyInput(message) match {
  	case LeveledMessage(msg, level) =>
  	map0(EF.effectOf(canLog.getLogger(level)(msg())))(_ => message)
  	}

[codecov]

Codecov Report

Merging #502 (a613c0a) into main (2865629) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #502   +/-   ##
=======================================
  Coverage   97.64%   97.64%           
=======================================
  Files          15       15           
  Lines         170      170           
  Branches        5        5           
=======================================
  Hits          166      166           
  Misses          4        4