search

pinojs / pino-pretty

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

fix: unescape \\ for levelKey #538

Closed eliw00d closed 2 weeks ago

eliw00d commented 2 weeks ago

closes #456

eliw00d commented 2 weeks ago

@mcollina Pino 9.0.0 dropped support for Node 14. Is that something that can be done here as well? I can switch to replace if not.

jsumners commented 2 weeks ago

Yes, we should drop 14. It'll be a new major, though.

eliw00d commented 2 weeks ago

Yes, we should drop 14. It'll be a new major, though.

Should I make that change in this PR? For example:

      matrix:
        node-version: [18.17, 20, 22]
        os: [ubuntu-latest]
        pino-version: [^9.0.0]
mcollina commented 2 weeks ago

@eliw00d send a fresh PR.

eliw00d commented 2 weeks ago

@eliw00d send a fresh PR.

Done.

eliw00d commented 2 weeks ago

@jsumners @mcollina Should be good to go now! 🎉