Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use logger context instead of var_export #283

Merged
merged 1 commit into from
Oct 4, 2017

Conversation

barryvdh
Copy link
Contributor

@barryvdh barryvdh commented Oct 4, 2017

Fixes #274

Doesn't var_export variables, but use the context option from the Log interface. This leaves the exporting to the logger.

@akerouanton akerouanton merged commit 1afa788 into KnpLabs:master Oct 4, 2017
@akerouanton
Copy link
Contributor

Thank you @barryvdh :)

@barryvdh barryvdh deleted the patch-1 branch October 4, 2017 19:55
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants