rcorp / standard-project-structure

A mission to improve developer productivity. Automate Everything ™ !
Apache License 2.0
5 stars 3 forks source link

add linters config file and Gruntfile.js #45

Closed neerajsachan closed 7 years ago

neerajsachan commented 7 years ago

we have split Gruntfile.js into multiple task using grunt plugin.

neerajsachan commented 7 years ago

I have change the configuration file of the stylelint.js , eslint.js and add the .lintignore file in generators/linter , add the html plugin in generators/linter, .eslintrc file and also add the "leslint-plugin-html" in the index.js file.

neerajsachan commented 7 years ago

Delete the Gruntfile.js from the generators/linter/templates/ , folder and also change the configuration file tslint.js ./.linterignore instead of ./.lintignore .

neerajsachan commented 7 years ago

change the tslint.js file ./.linterignore instead of ./.lintignore .

gaurav21r commented 7 years ago

@neerajsachan Please do:

  1. Update devDependencies to include minimum dependencies we need after converting linters to grunt plugins.
  2. Use yeoman's copyTpl to copy gruntfiles.
neerajsachan commented 7 years ago

@gaurav21r after installing grunt plugins we need the plugins grunt-eslint for JavaScript file , grunt-stylelint and stylelint-config-standard for the .css file , grunt-htmlhint for the .html file and the grunt-tslint and tslint-microsoft-contrib for the typescript file . But eslint plugin,i.e.,eslint-plugin-html is not included in .html file, although we are still working on it.

riteshe63 commented 7 years ago

@neerajsachan .eslintrc is in use for airbnb module and this module is not define, So please check again.

neerajsachan commented 7 years ago

@riteshe63 airbnb module use in .eslintrc but when install the grunt-eslint , airbnb module install automatically so we don't need to install airbnb module specially.

riteshe63 commented 7 years ago

Please check.

riteshe63 commented 7 years ago

@gaurav21r Add task ruuner to linter-generator.

riteshe63 commented 7 years ago

All done