All committers have signed the CLA.

I checkout this branch from master branch and send PR to development branch. Seems that makes those other commits there. Dose this matter?

Thanks for this contribution, I will try it out on a mac and eill merge this PR. No it does not matter that some other commits got introduced in it, that’s my fault not having merged the hot fix in dev)

OK. Thanks.

Actually there's an error in test case code when i run npm test on my mac:

Here's where error happen:

Sorry i just find out the commit message rules. And the commit in this PR dose not follow that. T_T
Maybe we could check the commit message by git hook? If you'd like, i'll send a PR for that by using huksy and commitlint.

I'm interested in this patch well—let me know if I can be of any use! @poppinlp were you able to fix the test issue on OSX? I imagine that the commit messages may be able to be fixed by squashing the commits when this PR is merged.

@devinivy OK. I will try to fix the test issue recently if i have time (busy work T_T).