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

Traversing Free structure to inject term logging #338

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@blast-hardcheese
Copy link
Collaborator

commented Jul 9, 2019

In an attempt to make it easier to debug either while developing or when running guardrail, this PR abuses the deferred nature of Free to dynamically inject logs every time there's a FlatMapped(Suspend(term), f), which is the construct that maps to calling out to a member of a Free algebra.

Abusing package membership is necessary because there are no public functions on Free that expose the tree itself, only handling individual terms, like mapK.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

@blast-hardcheese blast-hardcheese force-pushed the blast-hardcheese:injecting-logging branch from 06869e5 to 46d8a33 Jul 9, 2019

@blast-hardcheese blast-hardcheese force-pushed the blast-hardcheese:injecting-logging branch from 46d8a33 to 9f9d375 Jul 10, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.