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 common logger #69

Merged
merged 6 commits into from
Oct 10, 2023
Merged

Use common logger #69

merged 6 commits into from
Oct 10, 2023

Conversation

Siegrift
Copy link
Collaborator

@Siegrift Siegrift commented Oct 6, 2023

Because commons only had one functionality (the logger) it is now removed in favor of the logger from commons.

@Siegrift Siegrift requested a review from andreogle October 6, 2023 18:03
@Siegrift Siegrift self-assigned this Oct 6, 2023
Base automatically changed from eslint-unification to main October 9, 2023 09:21
@Siegrift Siegrift force-pushed the use-common-logger branch 2 times, most recently from d865f7f to a7b0b64 Compare October 9, 2023 09:39
@Siegrift Siegrift force-pushed the use-common-logger branch 3 times, most recently from b75e049 to 5b10549 Compare October 9, 2023 14:33
packages/api/src/schema.test.ts Show resolved Hide resolved
@Siegrift Siegrift merged commit e7051c4 into main Oct 10, 2023
4 checks passed
@Siegrift Siegrift deleted the use-common-logger branch October 10, 2023 06:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants