Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

one-line-logger as a executable program #23

Open
2 tasks done
erfanium opened this issue May 22, 2023 · 5 comments
Open
2 tasks done

one-line-logger as a executable program #23

erfanium opened this issue May 22, 2023 · 5 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@erfanium
Copy link

erfanium commented May 22, 2023

Prerequisites

  • I have written a descriptive issue title
  • I have searched existing issues to ensure the feature has not already been requested

馃殌 Feature Proposal

I would love to use one-line-logger like this.

node app.js | @fastify/one-line-logger

it it possible and in the scope of this repo?
i would like to contribute on it

Motivation

No response

Example

No response

@Eomm
Copy link
Member

Eomm commented May 23, 2023

Not sure if pino is sponsoring this approach as it is called "legacy" in its docs: https://github.com/pinojs/pino/blob/master/docs/transports.md#legacy-transports

Opinions @climba03003 ?

From my point of view, I like the pipe operator 馃憤馃徏

@erfanium
Copy link
Author

@Eomm
there's pino-pretty which is not legacy.
https://github.com/pinojs/pino-pretty

@Uzlopak
Copy link
Contributor

Uzlopak commented May 23, 2023

@mcollina
Copy link
Member

We should probably find a better term than "legacy" in the pino docs.

@climba03003
Copy link
Member

Opinions @climba03003 ?

We can go with simple script that pipe the stdin to Transform first.

Note that this feature is only useful in development.
Welcome for PR.

@Eomm Eomm added enhancement New feature or request good first issue Good for newcomers labels May 23, 2023
@erfanium erfanium mentioned this issue May 27, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants