Convert keywords to an array #61

pdehaan · 2016-09-01T22:48:39Z

Fixes #48

r? @jaredkerim

coveralls · 2016-09-01T22:50:33Z

Coverage remained the same at 100.0% when pulling 89f76fc on pdehaan:issue-48-redux into cef465e on mozilla:master.

pdehaan · 2016-09-02T18:26:36Z

parser.js

+        if (value.trim) {
+          return value.trim();
+        }
+        return value;


This was required because the keywords post-processor converts the String to an Array, which doesn't have a .trim() method, so it was blowing up.

jaredlockhart · 2016-09-02T19:08:04Z

r+

Convert keywords to an array

89f76fc

pdehaan mentioned this pull request Sep 1, 2016

Return keywords as array instead of string #49

Closed

pdehaan reviewed Sep 2, 2016
View reviewed changes

jaredlockhart merged commit 61c58cb into mozilla:master Sep 2, 2016

Provide feedback