Skip to content
This repository has been archived by the owner on Sep 23, 2024. It is now read-only.

Remove zk-evm/log dependancy #73

Merged
merged 2 commits into from
Feb 23, 2024

Conversation

goran-ethernal
Copy link
Contributor

@goran-ethernal goran-ethernal commented Feb 12, 2024

This PR removes the zk-evm/log dependency by implementing its logger (with a minimum set of functions) using the go.uber library.

Copy link
Contributor

@vcastellm vcastellm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Stefan-Ethernal Stefan-Ethernal force-pushed the CDK-13-Refactor-logging-implementation branch from ad693ec to 2971cad Compare February 23, 2024 11:35
Copy link

sonarcloud bot commented Feb 23, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@Stefan-Ethernal Stefan-Ethernal merged commit 8b2b965 into main Feb 23, 2024
5 of 6 checks passed
@Stefan-Ethernal Stefan-Ethernal deleted the CDK-13-Refactor-logging-implementation branch February 23, 2024 11:45
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants