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

Detect missing exports.lab in the cli runner #159

Merged
merged 5 commits into from Aug 15, 2014

Conversation

@geek
Copy link
Member

geek commented Aug 14, 2014

Fixes #157

lib/cli.js Outdated
@@ -86,6 +87,10 @@ internals.traverse = function (paths, options) {

scripts.push(pkg.lab);
}
else if (global._labScriptRun) {
options.output.write('Missing exports.lab found. Please fix and run again.');

This comment has been minimized.

Copy link
@hueniverse

hueniverse Aug 15, 2014

Member

Message is a bit cryptic. Need to be more explicit on how to fix it.

@hueniverse hueniverse added the bug label Aug 15, 2014
@hueniverse hueniverse self-assigned this Aug 15, 2014
@hueniverse hueniverse added this to the 4.0.3 milestone Aug 15, 2014
@geek

This comment has been minimized.

Copy link
Member Author

geek commented Aug 15, 2014

@hueniverse fixed the message

geek added 3 commits Aug 15, 2014
hueniverse added a commit that referenced this pull request Aug 15, 2014
Detect missing exports.lab in the cli runner
@hueniverse hueniverse merged commit 7e763d1 into hapijs:master Aug 15, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
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.