test: adds a console.log in call handling function

defi-wonderland / smock

The Solidity mocking library

MIT License

319 stars 40 forks source link

test: adds a console.log in call handling function #130

Closed hickscorp closed 2 years ago

hickscorp commented 2 years ago

Description A clear and concise description of the features you're adding in this pull request.

Metadata

Fixes #[Link to Issue]

wei3erHase commented 2 years ago

ey @hickscorp ! i really like the idea tbh, but it'd be nice to implement it with a proper logging system to set up when the user wants to receive those logs, and even filtering them out. i'd request it to be optional for the user to merge it, i'd suggest winston library for such, it'd be interesting to have!

hickscorp commented 2 years ago

But @wei3erHase I don't want to merge anything... I just want to instrument smock locally on my computer, but the only answer I got from you guys to make that work is to go through a canary release...

0xGorilla commented 2 years ago

This should be done via yarn link locally