SamVerschueren / clinton

Project style linter
MIT License
122 stars 9 forks source link

Opinionated ideas #88

Closed dy closed 7 months ago

dy commented 6 years ago

@SamVerschueren do you mind if I put some ideas to make clinton a single project linting tool? Please feel free to take them over or delete the issue.

sindresorhus commented 6 years ago

Make sure package has build script on prepublish, compiling code to ES5 or ES3 (to avoid publishing unbuilt source)

👎 ES2015 all the way. No need to compile down unless you use non-standard syntax.

Make sure .npmignore has redundant directories and files, like images, docs, etc.

This should be ensuring you specify a files field in package.json instead.

Make sure .gitattributes includes proper linguist config eg. a-vis/regl-line2d:.gitattributes@master

That is only needed the few times GitHub doesn't correctly detect it. In most cases it will.

dy commented 7 months ago

Not sure why I even started giving unsolicited advice.