jan/mustache.js - mustache.js - Gitea: Git with a cup of tea

Don't evaluate lambdas that return falsy values as an empty string. Issue: If I create a Mustache lambda to return numbers and I expect 0 to be of those numbers, I'll instead only receive an empty string for 0 because it evaluates to falsy. Fix: Check the result of the lambda to be undefined or null. Only then should it return the empty string.

13 år sedan

Start linting all test/ files as part of test script (#704) In an effort of ensuring consistent code style in test files as with the "production" source code, we should run eslint as part of the `$ npm test` script as well. Most of the related fixes was done by `eslint` using the `--fix` argument. Only special configuration tweaks for tests compared to the other source code, is to allow functions declaration without names. The rationale for allowing that in tests, is that the important reason we have them in the source code (proper stacktraces) aren't as important in test files.

6 år sedan

Don't evaluate lambdas that return falsy values as an empty string. Issue: If I create a Mustache lambda to return numbers and I expect 0 to be of those numbers, I'll instead only receive an empty string for 0 because it evaluates to falsy. Fix: Check the result of the lambda to be undefined or null. Only then should it return the empty string.

13 år sedan

Start linting all test/ files as part of test script (#704) In an effort of ensuring consistent code style in test files as with the "production" source code, we should run eslint as part of the `$ npm test` script as well. Most of the related fixes was done by `eslint` using the `--fix` argument. Only special configuration tweaks for tests compared to the other source code, is to allow functions declaration without names. The rationale for allowing that in tests, is that the important reason we have them in the source code (proper stacktraces) aren't as important in test files.

6 år sedan

Don't evaluate lambdas that return falsy values as an empty string. Issue: If I create a Mustache lambda to return numbers and I expect 0 to be of those numbers, I'll instead only receive an empty string for 0 because it evaluates to falsy. Fix: Check the result of the lambda to be undefined or null. Only then should it return the empty string.

13 år sedan

Start linting all test/ files as part of test script (#704) In an effort of ensuring consistent code style in test files as with the "production" source code, we should run eslint as part of the `$ npm test` script as well. Most of the related fixes was done by `eslint` using the `--fix` argument. Only special configuration tweaks for tests compared to the other source code, is to allow functions declaration without names. The rationale for allowing that in tests, is that the important reason we have them in the source code (proper stacktraces) aren't as important in test files.

6 år sedan

1234567

({
  number: function (text, render) {
    return function (text, render) {
      return +render(text);
    };
  }
});