dthree / vorpal

Node's framework for interactive CLIs
http://vorpal.js.org
MIT License
5.63k stars 278 forks source link

Feature/tslint #335

Closed slinkardbrandon closed 5 years ago

slinkardbrandon commented 5 years ago

This adds in TSLint and prettier configs to the project. The TSLint rules should give us a pretty decent idea of everything that needs to be converted to modern typescript, if we're rewriting I figured prettier rules wouldn't be a bad thing to put in place so we had some standards built in.

I'm not running lint checks in our ci or anything yet as there will be a ton of problems. I'm open for debate and answering questions on the rules I chose. I took a base from one of my personal projects and removed a few rules I thought wouldn't apply here.

Let me know what you think.

slinkardbrandon commented 5 years ago

Closing PR as all work for the typescript migration will take place in the Vorpaljs-reforge org until a PR can be made to the 2.0-ts branch here.