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

Allows Good to start with no reporters. #356

Merged
merged 1 commit into from Jun 4, 2015
Merged

Allows Good to start with no reporters. #356

merged 1 commit into from Jun 4, 2015

Conversation

@arb
Copy link
Contributor

arb commented Jun 4, 2015

Closes #345.

@arb arb added the feature label Jun 4, 2015
@arb arb added this to the 6.2.0 milestone Jun 4, 2015
monitor.stop();
done();
});

This comment has been minimized.

Copy link
@cjihrig

cjihrig Jun 4, 2015

Contributor

No blank line here.

@cjihrig

This comment has been minimized.

Copy link
Contributor

cjihrig commented Jun 4, 2015

One style nit. LGTM

Closes #345.
@arb arb force-pushed the arb:no-reporters branch from 7a8bad5 to d3d55a8 Jun 4, 2015
arb added a commit that referenced this pull request Jun 4, 2015
Allows Good to start with no reporters.
@arb arb merged commit b89b99c into hapijs:master Jun 4, 2015
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@arb arb deleted the arb:no-reporters branch Jun 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.