Omit files or directories with underscore on it (#44)

Author: James Kolce

index.js

@@ -242,10 +242,12 @@ function stdin() {
  */
 
 function renderFile(path, rootPath) {
-  var re = /\.(?:pug|jade)$/;
+  var isPug = /\.(?:pug|jade)$/;
+  var isIgnored = /([\/\\]_)|(^_)/;
+
   var stat = fs.lstatSync(path);
   // Found pug file
-  if (stat.isFile() && re.test(path)) {
+  if (stat.isFile() && isPug.test(path) && !isIgnored.test(path)) {
     // Try to watch the file if needed. watchFile takes care of duplicates.
     if (program.watch) watchFile(path, null, rootPath);
     if (program.nameAfterFile) {
@@ -269,7 +271,7 @@ function renderFile(path, rootPath) {
     else                     extname = '.html';
 
     // path: foo.pug -> foo.<ext>
-    path = path.replace(re, extname);
+    path = path.replace(isPug, extname);
     if (program.out) {
       // prepend output directory
       if (rootPath) {

test/index.js

@@ -103,6 +103,7 @@ function timing(testCase) {
  * Make temporary directories
  */
 rimraf.sync(t([]));
+mkdirp.sync(t(['_omittedDir']));
 mkdirp.sync(t(['depwatch']));
 mkdirp.sync(t(['inputs', 'level-1-1']));
 mkdirp.sync(t(['inputs', 'level-1-2']));
@@ -135,6 +136,28 @@ describe('miscellanea', function () {
       });
     });
   });
+  it('Omits files starting with an underscore', function (done) {
+    w('_omitted.pug', '.foo bar');
+    w('_omitted.html', '<p>output not written</p>');
+
+    run(['_omitted.pug'], function (err) {
+      if (err) return done(err);
+      var html = r('_omitted.html');
+      assert(html === '<p>output not written</p>');
+      done();
+    });
+  });
+  it('Omits directories starting with an underscore', function (done) {
+    w('_omittedDir/file.pug', '.foo bar');
+    w('_omittedDir/file.html', '<p>output not written</p>');
+
+    run(['--no-debug', '_omittedDir/file.pug'], function (err, stdout) {
+      if (err) return done(err);
+      var html = r('_omittedDir/file.html');
+      assert.equal(html, '<p>output not written</p>');
+      done();
+    });
+  });
 });
 
 describe('HTML output', function () {
@@ -335,12 +358,12 @@ describe('client JavaScript output', function () {
       return done();
     });
   });
-  it('--name-after-file _InPuTwIthWEiRdNaMME.pug', function (done) {
-    w('_InPuTwIthWEiRdNaMME.pug', '.foo bar');
-    w('_InPuTwIthWEiRdNaMME.js', 'throw new Error("output not written");');
-    run(['--no-debug', '--client', '--name-after-file', '_InPuTwIthWEiRdNaMME.pug'], function (err, stdout, stderr) {
+  it('--name-after-file ·InPuTwIthWEiRdNaMME.pug', function (done) {
+    w('·InPuTwIthWEiRdNaMME.pug', '.foo bar');
+    w('·InPuTwIthWEiRdNaMME.js', 'throw new Error("output not written");');
+    run(['--no-debug', '--client', '--name-after-file', '·InPuTwIthWEiRdNaMME.pug'], function (err, stdout, stderr) {
       if (err) return done(err);
-      var template = Function('', r('_InPuTwIthWEiRdNaMME.js') + ';return InputwithweirdnammeTemplate;')();
+      var template = Function('', r('·InPuTwIthWEiRdNaMME.js') + ';return InputwithweirdnammeTemplate;')();
       assert(template() === '<div class="foo">bar</div>');
       return done();
     });