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

Replace 'print' statements with logging #49

Open
trey-stafford opened this issue Jan 15, 2025 · 1 comment
Open

Replace 'print' statements with logging #49

trey-stafford opened this issue Jan 15, 2025 · 1 comment

Comments

@trey-stafford
Copy link
Member

We've made use of print a lot in this code, but we should be using logging. We will likely want to create some form of persistent logs for the OGDC in production.

I would suggest we use loguru. We use this on projects at NISDC and it helps to simplify/do away with the boilerplate that's otherwise necessary.

@mbjones
Copy link

mbjones commented Jan 15, 2025

+1 on simple logging config.

Note related discussion in PermafrostDiscoveryGateway/viz-staging#47 and a few other tickets in the libraries.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Todo
Development

No branches or pull requests

2 participants