pinojs / pino-pretty

🌲Basic prettifier for Pino log lines
MIT License
1.27k stars 150 forks source link

Add customColors to typescript definition for PrettyOptions #440

Closed Hufschmidt closed 1 year ago

Hufschmidt commented 1 year ago

See https://github.com/pinojs/pino-pretty/issues/439

Also contains minor cleanup of type test-file.


This assumes index.d.ts is managed manually and not autogenerated by some process as I didn't see any related script in package.json.

Also I've not worked with TSD so far, so I only added the missing example values for customLevels and customColors parameters to the options. A quick inverse check confirmed tests working as expected as they failed without adding customColors to the type definition. :+1:

coveralls commented 1 year ago

Pull Request Test Coverage Report for Build 5599626822


Totals Coverage Status
Change from base Build 5455898307: 0.0%
Covered Lines: 410
Relevant Lines: 410

💛 - Coveralls