Clean configuration and improve readme

Clean configuration, improve readme, write some necessary comments.

Author: fciancio

README.md

@@ -19,6 +19,23 @@ npm install -g yo
 
 Yeoman travels light. He didn't pack any generators when he moved in. You can think of a generator like a plug-in. You get to choose what type of application you wish to create, such as a Backbone application or even a Chrome extension.
 
+### Generator info
+
+The Angular Require Fullstack generator comes to speed up the scaffolding of projects using both AngularJS and RequireJS alongside with some must-have tools/plugins like file revving, i18n, minification of images and files among others. 
+
+####Why do we need a generator at all?
+Well, unless you like spending time creating the basic structure for each project you start, setting up its configuration and the tools/plugins/frameworks you're going to use, you will agree that those tasks are terrible repetitive, tough, long and on top of that, they have a ROI tending to zero. Using a generator will simplify most of these tasks in seconds.
+
+####Why to use this generator?
+We believe that both AngularJS and RequireJS are extremely usefull when building up web applications, and we wanted to use both of them on all of the web projects we develop, but we couldn't find a generator that did these while providing support to file revving and i18n, which we think must be present in any decent web application. 
+
+####Who built this generator?
+This generator was built by Monits. If you'd like to know more about Monits, go to [Monits](http://www.monits.com) or check out our [Blog](https://medium.com/monits-blog)
+
+![](monits_logo.png)
+
+### How to use it
+
 To install generator-angular-require-fullstack from npm, run:
 
 ```bash
@@ -30,20 +47,14 @@ Finally, initiate the generator:
 ```bash
 yo angular-require-fullstack
 ```
-
-### Generator info - Monits
-
-This generator was built by Monits. If you'd like to know more about Monits, go to [Monits](http://www.monits.com) or check out our [Blog](https://medium.com/monits-blog)
-
-![](monits_logo.png)
+It will guide you from there
 
 ### Getting To Know Yeoman
 
 Yeoman has a heart of gold. He's a person with feelings and opinions, but he's very easy to work with. If you think he's too opinionated, he can be easily convinced.
 
 If you'd like to get to know Yeoman better and meet some of his friends, [Grunt](http://gruntjs.com) and [Bower](http://bower.io), check out the complete [Getting Started Guide](https://github.com/yeoman/yeoman/wiki/Getting-Started).
 
-
 ## License
 
 MIT

app/index.js

@@ -51,13 +51,15 @@ module.exports = yeoman.generators.Base.extend({
 
   writing: {
     app: function () {
-      this.fs.copy(
+      this.fs.copyTpl(
         this.templatePath('_package.json'),
-        this.destinationPath('package.json')
+        this.destinationPath('package.json'),
+        this
       );
-      this.fs.copy(
+      this.fs.copyTpl(
         this.templatePath('_bower.json'),
-        this.destinationPath('bower.json')
+        this.destinationPath('bower.json'),
+        this
       );
 
       //Main files
@@ -166,8 +168,8 @@ module.exports = yeoman.generators.Base.extend({
         this.destinationPath('.editorconfig')
       );
       this.fs.copy(
-        this.templatePath('jshintrc'),
-        this.destinationPath('.jshintrc')
+        this.templatePath('.eslintrc'),
+        this.destinationPath('.eslintrc')
       );
     }
   },

app/templates/.eslintrc

@@ -0,0 +1,90 @@
+{
+    "env": {
+        "browser": true
+    },
+    "globals": {
+        "require": true,
+        "define": true,
+	"$": true,
+	"angular": true,
+	"moment": true
+    },
+    "rules": {
+        "block-scoped-var": 1,
+        "brace-style": [2, "1tbs"],
+        "camelcase": 2,
+        "comma-style": [2, "last"],
+        "consistent-this": [0, "self"],
+        "curly": 2,
+        "default-case": 2,
+        "dot-notation": 2,
+        "eqeqeq": 1,
+        "func-names": 0,
+        "func-style": 0,
+        "guard-for-in": 1,
+        "key-spacing": 1,
+        "max-nested-callbacks": [2, 5],
+        "new-cap": 2,
+        "new-parens": 1,
+        "no-alert": 2,
+        "no-array-constructor": 2,
+        "no-comma-dangle": 2,
+        "no-constant-condition": 1,
+        "no-control-regex": 2,
+        "no-debugger": 2,
+        "no-dupe-keys": 1,
+        "no-empty": 2,
+        "no-eval": 2,
+        "no-extra-bind": 1,
+        "no-extra-parens": 0,
+        "no-fallthrough": 1,
+        "no-floating-decimal": 1,
+        "no-func-assign": 0,
+        "no-implied-eval": 2,
+        "no-invalid-regexp": 2,
+        "no-labels": 2,
+        "no-lone-blocks": 1,
+        "no-lonely-if": 1,
+        "no-mixed-requires": [1, true],
+        "no-multi-spaces": 1,
+        "no-multi-str": 1,
+        "no-native-reassign": 2,
+        "no-nested-ternary": 2,
+        "no-new-object": 1,
+        "no-octal": 2,
+        "no-redeclare": 1,
+        "no-script-url": 2,
+        "no-sequences": 1,
+        "no-shadow-restricted-names": 0,
+        "no-space-before-semi": 2,
+        "no-spaced-func": 2,
+        "no-ternary": 0,
+        "no-underscore-dangle": 0,
+        "no-unreachable": 1,
+        "no-unused-expressions": 1,
+        "no-void": 1,
+        "no-wrap-func": 1,
+        "one-var": 0,
+        "quote-props": 0,
+        "quotes": [1, "single", "avoid-escape"],
+        "radix": 2,
+        "semi": 2,
+        "sort-vars": 0,
+        "space-after-keywords": [2, "always"],
+        "space-before-blocks": [1, "always"],
+        "space-in-brackets": [2, "never"],
+        "space-in-parens": [2, "never"],
+        "space-infix-ops": 2,
+        "space-return-throw-case": 2,
+        "space-unary-ops": [1, { "words": true, "nonwords": false }],
+        "spaced-line-comment": ["always", { exceptions: ["-", "#"]}],
+        "strict": 2,
+        "use-isnan": 2,
+        "valid-jsdoc": 0,
+        "valid-typeof": 1,
+        "vars-on-top": 0,
+        "wrap-iife": [2, "inside"],
+        "wrap-regex": 2,
+        "yoda": [1, "never"]
+    }
+}

app/templates/Gruntfile.js

@@ -1,3 +1,5 @@
+/* eslint-env node */
+
 module.exports = function (grunt) {
   'use strict';
   require('load-grunt-tasks')(grunt);
@@ -9,8 +11,6 @@ module.exports = function (grunt) {
     dist: 'dist'
   };
 
-  
-
   grunt.initConfig({
     yeoman: appConfig,
     connect: {
@@ -471,7 +471,7 @@ module.exports = function (grunt) {
     grunt.task.run(['serve:' + target]);
   });
 
-  grunt.registerTask('build', 'Compiles app for production or release candidate', function (target) {
+  grunt.registerTask('build', 'Compiles app for production or release candidate', function () {
     grunt.task.run([
       'clean:dist',
       // copy stylesheets, in: app/styles/ out: .tmp/styles
@@ -511,4 +511,4 @@ module.exports = function (grunt) {
     'newer:eslint',
     'build'
   ]);
-};
+};

app/templates/_bower.json

@@ -1,5 +1,5 @@
 {
-  "name": "package",
+  "name": "<%= appname %>",
   "version": "0.0.0",
   "dependencies": {
     "angular": "~1.2.0",
@@ -8,12 +8,10 @@
     "bootstrap-sass-official": "~3.2.0",
     "angular-route": "~1.2.0",
     "moment": "~2.8.3",
-    "underscore": "~1.7.0",
-    "requirejs": "~2.1.11"
+    "requirejs": "~2.1.11",
+    "angular-translate": "~2.4.2"
   },
   "devDependencies": {
-    "jQuery.dotdotdot": "~1.6.16",
-    "angular-translate": "~2.4.2",
     "bootstrap": "~3.3.1"
   }
 }

app/templates/_package.json

@@ -1,5 +1,5 @@
 {
-  "name": "package",
+  "name": "<%= appname %>",
   "version": "0.0.0",
   "dependencies": {},
   "devDependencies": {

app/templates/jshintrc

@@ -9,7 +9,7 @@ define(['routes',
 		'use strict';
 		var <%= appname %> = angular.module('<%= appname %>', [
 			'ngRoute',
-			'pascalprecht.translate',
+			'pascalprecht.translate'
 		]);
 		<%= appname %>
 			.config(

app/templates/scripts/_app.js

@@ -1,3 +1,4 @@
+/* global paths */
 require.config({
     baseUrl: '/scripts',
     paths: {
@@ -24,7 +25,7 @@ require.config({
         moment: '../../bower_components/moment/moment',
         requirejs: '../../bower_components/requirejs/require',
         underscore: '../../bower_components/underscore/underscore',
-        'angular-translate': '../../bower_components/angular-translate/angular-translate',
+        'angular-translate': '../../bower_components/angular-translate/angular-translate'
     },
     shim: {
         angular: {
@@ -57,7 +58,7 @@ require.config({
             deps: [
                 'angular'
             ]
-        },
+        }
     },
     packages: [
 

app/templates/scripts/_build.js

@@ -1 +1,3 @@
+/* eslint-disable */
+//Used by filerev grunt task to output the creation of require-js compatible path mappings
 var paths = undefined;