Skip to content

Conversation

@hofan41
Copy link
Contributor

@hofan41 hofan41 commented Jul 1, 2015

Closes #285

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add a comment to the top of the file that indicates that multiple is required further down. Just easier to look at the module section to find dependencies

// ('./multiple') loaded below

@geek
Copy link
Member

geek commented Jul 1, 2015

@hofan41 Please update the readme with information on how to use this enhancement

@geek geek added the feature New functionality or improvement label Jul 1, 2015
@geek geek self-assigned this Jul 1, 2015
@geek
Copy link
Member

geek commented Jul 2, 2015

@hofan41 really well done, this is awesome!

@geek geek added this to the 5.12.0 milestone Jul 2, 2015
geek added a commit that referenced this pull request Jul 2, 2015
Multiple reporters and multiple outputs
@geek geek merged commit 83acb83 into hapijs:master Jul 2, 2015
@hofan41
Copy link
Contributor Author

hofan41 commented Jul 2, 2015

Glad I could help! 👍

@hofan41 hofan41 deleted the rebase-9223fd8 branch July 3, 2015 23:07
@lock
Copy link

lock bot commented Jan 9, 2020

This thread has been automatically locked due to inactivity. Please open a new issue for related bugs or questions following the new issue template instructions.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

feature New functionality or improvement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use reporters from json, support multiple reports

2 participants