The new tmpl does not pass all the riot tests about tmpl HOT 6 CLOSED

I will

from tmpl.

@GianlucaGuarini , I push to the master brach this time, next PRs will go to the dev branch.
This version will pass all text, except:
https://github.com/riot/riot/blob/2.3.0/test/specs/tmpl.js#L133
expect(render('{ loading: !nonExistingVar.length }')).to.equal('loading')
Please comment that line, or change 'loading' to the empty string.

This change in the behavior of shorthands was for consistency with the evaluation of other expressions, now property accessors for undefined variables generate error, so that the resulting expression is blank.

There's a new folder ./src with commented code, I'm using ./rmcomms for cleanup the resulting file ./lib/index.js and conditionally include debugging code.
This is called by the Makefile, and it should be transparent to other parts of riot.

Note:
I'm disappointed with eslint 1.0, misaligned variables are poorly readable for me and sometimes is needed use extra parentheses to increase readability, so I returned to the previous version, but if there are problems I will use the new one.

from tmpl.

@aMarCruz well done! I am not sure we need 2 folders src and lib probably you can just use dist for the output like @cognitom does for riot-route https://github.com/riot/route/tree/master/dist

from tmpl.

Thanks.
for partial tools like tmpl I think 'lib' seems appropriate, anyway, /dist is a good name.
Please review this tree:

\                  // <-- root, with copy of tmpl.js?
\src               // <-- commented source files
\dist              // <-- tmpl.js , riot-tmpl.js or index.js ? with minified versions.
\test

from tmpl.

I prefer the riot-route folder structure:

\lib   // <-- commented files
\dist
   tmpl.riot.js <-- no umd
   tmpl.min.js <-- minified
   tmpl.js  <-- umd wrappers

Remember also to change in package.json the path to the UMD release https://github.com/riot/route/blob/master/package.json#L5

from tmpl.

this issue got solved

from tmpl.