Keep everything as TypeScript, no longer create a dist.

scatcher · Jan 10, 2017 · 5323137 · 5323137
1 parent f9f81c6
commit 5323137
Show file tree

Hide file tree

Showing 5 changed files with 22 additions and 451 deletions.
diff --git a/dist/angular-point-group-manager.js b/dist/angular-point-group-manager.js
diff --git a/package.json b/package.json
@@ -1,8 +1,7 @@
 {
   "name": "angular-point-group-manager",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "description": "angular-point tool to manage SharePoint users and groups.",
-  "main": "dist/angular-point-group-manager.js",
   "directories": {
     "test": "test"
   },
@@ -26,7 +25,6 @@
     "@types/source-map": "~0.5.0",
     "@types/toastr": "^2.1.32",
     "awesome-typescript-loader": "^3.0.0-beta.17",
-    "codelyzer": "^2.0.0-beta.4",
     "tslint": "^4.3.1",
     "tslint-loader": "^3.3.0",
     "typescript": "~2.1.1",

diff --git a/src/groupManager.ts b/src/groupManager.ts
@@ -5,7 +5,6 @@ import {DataContainer} from './dataContainer';
 import {IXMLGroup, IXMLUser, DataService} from 'angular-point';
 
 export class GroupManagerController {
-
     static $inject = ['NgTableParams', '$filter', '$timeout', '$q', 'apDataService'];
     activeTab = 'Users';
     assignedOptions = [];
@@ -21,12 +20,12 @@ export class GroupManagerController {
     users = new DataContainer();
     usersTable: Object;
 
-    constructor(
-        private NgTableParams,
-        private $filter: ng.IFilterService,
-        private $timeout: ng.ITimeoutService,
-        private $q: ng.IQService,
-        private apDataService: DataService) { }
+    constructor(private NgTableParams,
+                private $filter: ng.IFilterService,
+                private $timeout: ng.ITimeoutService,
+                private $q: ng.IQService,
+                private apDataService: DataService) {
+    }
 
     $onInit() {
 
@@ -52,7 +51,7 @@ export class GroupManagerController {
         data.assigned.length = 0;
         data.selectedAssigned.length = 0;
 
-        
+
         _.each(data.all, item => {
             if (_.indexOf(map, item.ID) > -1) {
                 // Already assigned
@@ -157,7 +156,7 @@ export class GroupManagerController {
         }).then((response: IXMLUser[]) => {
             // Assume that valid users all have email addresses and services/groups don't
             _.each(response, user => this.users.all.push(user));
-            
+
             deferred.resolve(this.users.all);
         });
         return deferred.promise;
@@ -173,7 +172,7 @@ export class GroupManagerController {
      */
     mergeGroups() {
         this.updatePermissions('AddUserToGroup', this.users.assigned, [this.targetGroup])
-            .then( (promiseArray) => {
+            .then((promiseArray) => {
                 toastr.success(promiseArray.length + ' users successfully merged.');
                 // Reset dropdowns to empty
                 this.sourceGroup = undefined;
@@ -243,7 +242,7 @@ export class GroupManagerController {
                         operation: operation, // AddUserToGroup || RemoveUserFromGroup'
                         groupName: group.Name,
                         userLoginName: user.LoginName
-                    }).then( (response) => {
+                    }).then((response) => {
                         deferred.resolve(response);
                     });
 
@@ -256,7 +255,7 @@ export class GroupManagerController {
             this.groups.clearSelected();
 
             // Resolved when all promises complete
-            this.$q.all(queue).then( (responses) => {
+            this.$q.all(queue).then((responses) => {
                 toastr.success(operation === 'AddUserToGroup' ?
                     'User successfully added' :
                     'User successfully removed');

diff --git a/tsconfig.json b/tsconfig.json
@@ -1,28 +1,20 @@
 {
   "compilerOptions": {
-    "noImplicitAny": false,
     "module": "es2015",
     "target": "es5",
-    "emitDecoratorMetadata": true,
+    "noImplicitAny": false,
+    "sourceMap": true,
     "experimentalDecorators": true,
+    "emitDecoratorMetadata": true,
     "declaration": true,
     "moduleResolution": "node",
-    "types": [
-      "hammerjs",
-      "jasmine",
-      "node"
-    ],
     "lib": ["es2015", "dom"]
   },
   "typeRoots": [
     "./node_modules/@types"
   ],
   "exclude": [
-    "out",
     "node_modules",
-    "dist",
-    "dev",
-    "reports"
-  ],
-  "version": "2.0.6"
+    "dist"
+  ]
 }
diff --git a/tslint.json b/tslint.json
@@ -1,7 +1,4 @@
 {
-  "rulesDirectory": [
-    "node_modules/codelyzer"
-  ],
   "rules": {
     "class-name": true,
     "comment-format": [
@@ -16,6 +13,7 @@
       "spaces"
     ],
     "label-position": true,
+    //    "label-undefined": true,
     "max-line-length": [
       true,
       140
@@ -38,6 +36,7 @@
     ],
     "no-construct": true,
     "no-debugger": true,
+    //    "no-duplicate-key": true,
     "no-duplicate-variable": true,
     "no-empty": false,
     "no-eval": true,
@@ -47,6 +46,9 @@
     "no-switch-case-fall-through": true,
     "no-trailing-whitespace": true,
     "no-unused-expression": true,
+    //    "no-unused-parameters": true,
+    //    "no-unused-locals": true,
+    //    "no-unreachable": true,
     "no-use-before-declare": true,
     "no-var-keyword": true,
     "object-literal-sort-keys": false,