-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: logger #27
feat: logger #27
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏻
apps/processing/.dockerignore
Outdated
.git | ||
.gitignore | ||
*.md | ||
generated |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will we have a .env in this dir?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we won't
isNativeError(error) ? error.stack : undefined, | ||
); | ||
const errorMessage = | ||
`Coingecko API error: failed to fetch token price` + |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
maybe leave a blank space after price so it's separated from the stringified error
The merge-base changed after approval.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good to go for me after addressing beebs comments, thanks for the logger btw 😄
🤖 Linear
Closes GIT-116
Description
Replace
console
calls with shared logger.Checklist before requesting a review