Could punch either 1) ignore files in template directory whose names begin with '.' (d

Repro w/ punch @ 0.1.2 Simple setup: <div class="snippet-clipboa

Files in template dir w/ no registered renderer cause error about punch HOT 3 CLOSED

johnvh commented on July 18, 2024

Files in template dir w/ no registered renderer cause error

from punch.

Comments (3)

johnvh commented on July 18, 2024

Repro w/ punch @ 0.1.2

Simple setup:

$ find .
.
./config.json
./contents
./contents/index.json
./public
./public/index.html
./templates
./templates/index.mustache

$ cat contents/index.json 
{"title": "demo"}

$ cat templates/index.mustache 
<h1>{{title}}</h1>

$ punch .
Generating Site...
Created public/index.html

Simulate editing index.mustache with vim by creating a dummy swap file:

$ touch templates/.index.mustache.swp

$ find . -iname \*.swp
./templates/.index.mustache.swp

$ punch .
Generating Site...

/usr/local/lib/node_modules/punch/lib/punch.js:32
    return (new self.registeredRenderers[content_type]); 
            ^
TypeError: undefined is not a function
    at Object.rendererFor (/usr/local/lib/node_modules/punch/lib/punch.js:32:13)
    at Object.fetchAndRender (/usr/local/lib/node_modules/punch/lib/punch.js:258:25)
    at /usr/local/lib/node_modules/punch/lib/punch.js:348:20
    at Array.forEach (native)
    at Object.oncomplete (/usr/local/lib/node_modules/punch/lib/punch.js:344:15)

from punch.

laktek commented on July 18, 2024

Hi John,

Thanks for the detailed ticket on the issue. I posted a fix to solve this issue. If you upgrade Punch (0.1.3) from NPM it should contain the fix.

However, please note still the .swp files are being copied to the output directory. You may avoid them from publishing by adding them to the .gitignore file (assuming you are using a SCM to publish your site) or use a bash script to prune them from the output directory.

from punch.

johnvh commented on July 18, 2024

Thanks for the quick turnaround!

from punch.

Recommend Projects