Skip to content

Commit

Permalink
Remove the success message
Browse files Browse the repository at this point in the history
Adopt 'No News is Good News' philosophy, which only errors are reports. You can keep the success message displaying by using --verbose.
  • Loading branch information
RobLoach committed Jan 3, 2017
1 parent 9a03c78 commit 6c4c491
Show file tree
Hide file tree
Showing 5 changed files with 11 additions and 12 deletions.
7 changes: 3 additions & 4 deletions gh-pages/api/master/internals/lib_kss.js.html
Expand Up @@ -19,11 +19,11 @@

<h1 class="page-title">Source: lib/kss.js</h1>








<section>
<article>
<pre class="prettyprint source linenums"><code>'use strict';
Expand Down Expand Up @@ -179,9 +179,8 @@ <h1 class="page-title">Source: lib/kss.js</h1>
return builder.build(styleGuide);
}).then(styleGuide => {
if (builder.getOptions('verbose')) {
builder.log('');
builder.log('Style guide build completed successfully!');
}
builder.log('Style guide build completed successfully!');
return Promise.resolve(styleGuide);
});
}).catch(error => {
Expand Down
7 changes: 3 additions & 4 deletions gh-pages/api/master/lib_kss.js.html
Expand Up @@ -19,11 +19,11 @@

<h1 class="page-title">Source: lib/kss.js</h1>








<section>
<article>
<pre class="prettyprint source linenums"><code>'use strict';
Expand Down Expand Up @@ -179,9 +179,8 @@ <h1 class="page-title">Source: lib/kss.js</h1>
return builder.build(styleGuide);
}).then(styleGuide => {
if (builder.getOptions('verbose')) {
builder.log('');
builder.log('Style guide build completed successfully!');
}
builder.log('Style guide build completed successfully!');
return Promise.resolve(styleGuide);
});
}).catch(error => {
Expand Down
3 changes: 1 addition & 2 deletions lib/kss.js
Expand Up @@ -151,9 +151,8 @@ const kss = function(options) {
return builder.build(styleGuide);
}).then(styleGuide => {
if (builder.getOptions('verbose')) {
builder.log('');
builder.log('Style guide build completed successfully!');
}
builder.log('Style guide build completed successfully!');
return Promise.resolve(styleGuide);
});
}).catch(error => {
Expand Down
3 changes: 2 additions & 1 deletion test/fixtures/cli-option-config-source-array.json
Expand Up @@ -2,5 +2,6 @@
"//" : "A JSON comment",
"source" : ["with-include", "missing-homepage"],
"destination" : "../output/nested",
"builder" : "builder"
"builder" : "builder",
"verbose" : true
}
3 changes: 2 additions & 1 deletion test/test_kss.js
Expand Up @@ -133,7 +133,8 @@ describe('kss object API', function() {
it('should warn if homepage content is not found', function() {
return testKss({
source: helperUtils.fixtures('missing-homepage'),
destination: 'test/output/nested'
destination: 'test/output/nested',
verbose: true
}).then(function(result) {
expect(result.error).to.not.exist;
expect(result.stdout).to.include(noHomepageWarning);
Expand Down

0 comments on commit 6c4c491

Please sign in to comment.