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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use @graphile/logger for logging #2154

Open
benjie opened this issue Aug 16, 2024 · 1 comment
Open

Use @graphile/logger for logging #2154

benjie opened this issue Aug 16, 2024 · 1 comment

Comments

@benjie
Copy link
Member

benjie commented Aug 16, 2024

Users should be able to use their own logging framework - we shouldn't be outputting to console. We already have the @graphile/logger abstraction, we should use this.

Needs integration from the preset all the way through every package.

Needs to be done before V5 goes out, I think, or at least we need to know how it will be done (to ensure e.g. graphile-config has support). We could make it a built-in feature of graphile-config TBH, everything that needs it will need logging of some kind.

@github-project-automation github-project-automation bot moved this to 🌳 Triage in V5.0.0 Aug 16, 2024
@benjie benjie moved this from 🌳 Triage to 🦉 Owl in V5.0.0 Aug 16, 2024
@jemgillam
Copy link
Contributor

This issue also pairs well with the idea of a diagnostics system

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: 🦉 Owl
Development

No branches or pull requests

2 participants