no-context / moo

Optimised tokenizer/lexer generator! 🐄 Uses /y for performance. Moo.
BSD 3-Clause "New" or "Revised" License
817 stars 65 forks source link

Fix unicode checks for fallbacks with test #125

Closed moranje closed 5 years ago

moranje commented 5 years ago

Test passes. Also I noticed two things when I cloned project from scratch:

{
  "printWidth": 80,
  "tabWidth": 2,
  "useTabs": false,
  "semi": false,
  "singleQuote": true,
  "trailingComma": "es5",
  "bracketSpacing": false,
  "arrowParens": "avoid"
}

I'd be willing to create pull-requests for these changes if you would welcome these.

tjvr commented 5 years ago

Yeah, NPM always seems to have audit failures because I don't upgrade dependencies very often. Which is silly, since Moo itself has no dependencies 🙂 A PR to upgrade the deps would be most welcome.

I've thought about adding prettier before, and possibly have a branch somewhere. It might be worth it (I really like go fmt, which it appears to emulate).

I'll try and review this PR later, thanks 😊

Sent with GitHawk