Merge pull request #112 from jakemmarsh/fix-ng-annotate

add ngInject prologues and documentation, remove mangle option

Author: jakemmarsh

README.md

@@ -77,7 +77,16 @@ export default {
 
 ##### Dependency injection
 
-Dependency injection is carried out with the `ng-annotate` library. The Gulp tasks will take care of injecting any dependencies, requiring you only to specify the dependencies within the function call and nothing more.
+Dependency injection is carried out with the `ng-annotate` library. In order to take advantage of this, a simple directive prologue of the format:
+
+```js
+function MyService($http) {
+  'ngInject';
+  ...
+}
+```
+
+needs to be added at the very beginning of any Angular functions/modules. The Gulp tasks will then take care of adding any dependency injection, requiring you to only specify the dependencies within the function parameters and nothing more.
 
 ---
 

app/js/filters/example.js

@@ -1,9 +1,11 @@
 'use strict';
 
 function ExampleFilter() {
+
   return function(input) {
     return input.replace(/keyboard/ig,'leopard');
   };
+
 }
 
 export default {

app/js/main.js

@@ -29,4 +29,6 @@ angular.module('app').config(require('./on_config'));
 
 angular.module('app').run(require('./on_run'));
 
-angular.bootstrap(document, ['app']);
+angular.bootstrap(document, ['app'], {
+  strictDi: true
+});

app/js/on_config.js

@@ -1,6 +1,7 @@
 'use strict';
 
 function OnConfig($stateProvider, $locationProvider, $urlRouterProvider) {
+  'ngInject';
 
   $locationProvider.html5Mode(true);
 

app/js/on_run.js

@@ -1,6 +1,7 @@
 'use strict';
 
 function OnRun($rootScope, AppSettings) {
+  'ngInject';
 
   // change page title based on state
   $rootScope.$on('$stateChangeSuccess', (event, toState) => {

app/js/services/example.js

@@ -1,6 +1,7 @@
 'use strict';
 
 function ExampleService($http) {
+  'ngInject';
 
   const service = {};
 

gulp/tasks/browserify.js

@@ -58,7 +58,6 @@ function buildScript(file) {
       .pipe(gulpif(createSourcemap, buffer()))
       .pipe(gulpif(createSourcemap, sourcemaps.init()))
       .pipe(gulpif(global.isProd, streamify(uglify({
-        mangle: false,
         compress: { drop_console: true }
       }))))
       .pipe(gulpif(createSourcemap, sourcemaps.write('./')))

package.json

@@ -1,6 +1,6 @@
 {
   "name": "angularjs-gulp-browserify-boilerplate",
-  "version": "1.0.3",
+  "version": "1.1.0",
   "author": "Jake Marsh <jakemmarsh@gmail.com>",
   "description": "Boilerplate using AngularJS, SASS, Gulp, and Browserify while also utilizing best practices.",
   "repository": {