search

pinojs / pino-elasticsearch

🌲 load pino logs into Elasticsearch
MIT License
179 stars 67 forks source link

Allow opType to be passed to cli #193

Closed johan-liljegren-visma closed 3 days ago

johan-liljegren-visma commented 2 months ago

Make it possible to pass --opType to the cli.

mcollina commented 1 month ago

Thanks for opening a PR! Can you please add a unit test to https://github.com/pinojs/pino-elasticsearch/blob/master/test/cli.test.js?

johan-liljegren-visma commented 1 month ago

I've added a test checking that the param is passed to the client, is this enough?