Skip to content

Add log-hook.#11

Open
kenrestivo wants to merge 1 commit intojoegallo:masterfrom
kenrestivo:master
Open

Add log-hook.#11
kenrestivo wants to merge 1 commit intojoegallo:masterfrom
kenrestivo:master

Conversation

@kenrestivo
Copy link
Copy Markdown

Add a log-hook for logging or otherwise tracking tries as they are in progress.

@kenrestivo
Copy link
Copy Markdown
Author

I just realized due to the precise way bruce handles true/false return values, could hack around this by using (abusing?) error-hook, i.e.

{:error-hook #(log/debug (.getMessage %)}

It gets a little complected if I have to actually do error-checking in the hook too, but it's manageable, so I don't really need this mod. But it'd be nice.

@joegallo
Copy link
Copy Markdown
Owner

Yeah, I've used that trick before. I rather like your solution, though, so I'll get this merged and released soonish.

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