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

Add verbose option. #27

Closed
wants to merge 2 commits into from
Closed

Conversation

andyburke
Copy link
Contributor

No description provided.

@anthonyshort
Copy link
Owner

This is cool, but I want to see if we can just emit some events and use them as hooks instead of putting the logging throughout the code.

So we can use it something like this:

builder.use(verboseLogging);

And the builder will emit events like start, file, end etc. that this can hook into. The main reason is just so that it decouples the logging.

@andyburke
Copy link
Contributor Author

Could you point me at an example? I'm fine with it being implemented differently, but I definitely missed having the output so I did the quickest thing to get it back.

@anthonyshort
Copy link
Owner

Yeah no problem, I'll take a look at emitting some events and providing an example

@kewah
Copy link
Collaborator

kewah commented Apr 22, 2014

Verbose option has been added in the version 0.5.0

@kewah kewah closed this Apr 22, 2014
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.

None yet

3 participants