fix: math filters now return number, resolves #110

* fix linting
* numbers are no longer fixed:
    i.e. {{0.3 | minus 0.1}} now renders as 0.19999
    Stick to JavaScript number behavior, see: 8.0.0

src/builtin/filters/math.ts

@@ -3,24 +3,12 @@ export default {
   'ceil': (v: number) => Math.ceil(v),
   'divided_by': (v: number, arg: number) => v / arg,
   'floor': (v: number) => Math.floor(v),
-  'minus': bindFixed((v: number, arg: number) => v - arg),
-  'modulo': bindFixed((v: number, arg: number) => v % arg),
+  'minus': (v: number, arg: number) => v - arg,
+  'modulo': (v: number, arg: number) => v % arg,
   'round': (v: number, arg: number = 0) => {
     const amp = Math.pow(10, arg)
     return Math.round(v * amp) / amp
   },
-  'plus': bindFixed((v: number, arg: number) => Number(v) + Number(arg)),
+  'plus': (v: number, arg: number) => Number(v) + Number(arg),
   'times': (v: number, arg: number) => v * arg
 }
-
-function getFixed (v: number) {
-  const p = String(v).split('.')
-  return (p.length > 1) ? p[1].length : 0
-}
-
-function bindFixed (cb: (v: number, arg: number) => number) {
-  return (l: number, r: number) => {
-    const f = Math.max(getFixed(l), getFixed(r))
-    return cb(l, r).toFixed(f)
-  }
-}

src/drop/drop.ts

@@ -1,7 +1,5 @@
-import { deprecate } from '../util/deprecate'
-
 export abstract class Drop {
-  valueOf(): any {
+  valueOf (): any {
     return undefined
   }
 

src/render/syntax.ts

@@ -64,8 +64,8 @@ export function parseExp (exp: string, scope: Scope): any {
   }
 
   if ((match = exp.match(lexical.rangeLine))) {
-    const low = parseValue(match[1], scope)
-    const high = parseValue(match[2], scope)
+    const low = evalValue(match[1], scope)
+    const high = evalValue(match[2], scope)
     return range(low, high + 1)
   }
 

src/util/deprecate.ts

@@ -1,7 +1,7 @@
 const reported:{[key: string]: boolean} = {}
 
-export function deprecate(msg: string, issue: number) {
+export function deprecate (msg: string, issue: number) {
   if (reported[msg]) return
   console.warn(msg + ` See: https://github.com/harttle/liquidjs/issues/${issue}`)
   reported[msg] = true
-}
+}

src/util/underscore.ts

@@ -96,21 +96,7 @@ export function isObject (value: any): value is object {
   return value !== null && (type === 'object' || type === 'function')
 }
 
-/*
- * A function to create flexibly-numbered lists of integers,
- * handy for each and map loops. start, if omitted, defaults to 0; step defaults to 1.
- * Returns a list of integers from start (inclusive) to stop (exclusive),
- * incremented (or decremented) by step, exclusive.
- * Note that ranges that stop before they start are considered to be zero-length instead of
- * negative — if you'd like a negative range, use a negative step.
- */
-export function range (start: number, stop?: number, step?: number) {
-  if (stop === undefined) {
-    stop = start
-    start = 0
-  }
-  step = step || 1
-
+export function range (start: number, stop: number, step: number = 1) {
   const arr: number[] = []
   for (let i = start; i < stop; i += step) {
     arr.push(i)