env_logger icon indicating copy to clipboard operation
env_logger copied to clipboard

feat(fmt)!: Generalize custom formatter to 'RecordFormat'

Open epage opened this issue 9 months ago • 2 comments

This breaks type inference for basic custom formatters

epage avatar Apr 01 '25 19:04 epage

Pull Request Test Coverage Report for Build 14844590081

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.4%) to 43.76%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/fmt/mod.rs 0 1 0.0%
<!-- Total: 0 1
Files with Coverage Reduction New Missed Lines %
src/fmt/mod.rs 4 62.5%
<!-- Total: 4
Totals Coverage Status
Change from base Build 14242549930: -0.4%
Covered Lines: 263
Relevant Lines: 601

💛 - Coveralls

coveralls avatar Apr 01 '25 19:04 coveralls

I have tested a similar change for #363 and i recommend against it.

Too much type info annoyance. I would instead recommend having two separate functions, one for the common case (FormatFn) and one for special needs (RecordFormat)

djugei avatar May 12 '25 13:05 djugei