pinojs / pino-pretty

🌲Basic prettifier for Pino log lines
MIT License
1.25k stars 147 forks source link

feat: dont print escape backslash on strings containing escaped backslashes #401

Closed ademarsj closed 1 year ago

ademarsj commented 1 year ago

Closes #378 , I'm not pretty sure about the way that I made it, but I think creating a function to do exactly that line of code is unnecessary.

mcollina commented 1 year ago

Could you fix it in Node v14? Otherwise it would need to wait until we bump the major to drop node v14.

ademarsj commented 1 year ago

Working on that.

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 4189532735


Totals Coverage Status
Change from base Build 4144067647: 0.0%
Covered Lines: 408
Relevant Lines: 408

💛 - Coveralls
ademarsj commented 1 year ago

Done, passing through all tests.

ademarsj commented 1 year ago

Grammar issues fixed.

ntzm commented 1 year ago

Thanks 🎉