avoid executing costly With if noop logger

[noboruma]

Prior to that change, Noop logger would execute With function for no benefits.

When running rule evaluation at high scale, the number of memory allocation sky rockets (~300MB / min at 1k RPS). This change proposes to make sure the logger is active before executing costly With function.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.13%. Comparing base (4ff1f76) to head (024d6be).
Report is 18 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1015      +/-   ##
==========================================
+ Coverage   82.72%   83.13%   +0.41%     
==========================================
  Files         162      162              
  Lines        9080     7556    -1524     
==========================================
- Hits         7511     6282    -1229     
+ Misses       1319     1026     -293     
+ Partials      250      248       -2     

Flag	Coverage Δ
default	83.13% <100.00%> (+5.30%)	⬆️
examples	83.13% <100.00%> (+56.70%)	⬆️
ftw	83.13% <100.00%> (+35.77%)	⬆️
ftw-multiphase	83.13% <100.00%> (+33.59%)	⬆️
tinygo	83.13% <100.00%> (+7.73%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[jcchavezs]

Looks very reasonable to me.

[jcchavezs]

LGTM