refactor: remove unnecessary eslint-disable

Author: rocwind

src/core/services/app-manager.ts

@@ -154,7 +154,6 @@ class AppManager {
                                 if (_.eq(u.get('id'), currentUid)) {
                                     isCurrentAccount = true;
                                 }
-                                // eslint-disable-next-line no-param-reassign
                                 allCol[u.get('email')] = {
                                     permission: collaborator.get('roles'),
                                     isCurrentAccount,

src/core/services/client-manager.ts

@@ -165,7 +165,6 @@ class ClientManager {
         appVersion: string,
         label: string,
         packageHash: string,
-        // eslint-disable-next-line @typescript-eslint/no-unused-vars
         clientUniqueId: string,
         logger: Logger,
     ) {

src/core/services/collaborators-manager.ts

@@ -18,7 +18,6 @@ class CollaboratorsManager {
                         data,
                         (result, value) => {
                             result.uids.push(value.uid);
-                            // eslint-disable-next-line no-param-reassign
                             result.colByUid[value.uid] = value;
                             return result;
                         },
@@ -38,7 +37,6 @@ class CollaboratorsManager {
                             if (!_.isEmpty(coInfo.colByUid[value.id])) {
                                 permission = coInfo.colByUid[value.id].roles;
                             }
-                            // eslint-disable-next-line no-param-reassign
                             result[value.email] = { permission };
                             return result;
                         },

src/core/services/deployments-manager.ts

@@ -104,7 +104,6 @@ class DeploymentsManager {
                             return _.reduce(
                                 diffs,
                                 (result, v) => {
-                                    // eslint-disable-next-line no-param-reassign
                                     result[v.diff_against_package_hash] = {
                                         size: v.diff_size,
                                         url: getBlobDownloadUrl(v.diff_blob_url),

src/models/deployments.ts

@@ -49,7 +49,6 @@ export function generateDeploymentsLabelId(deploymentId: number) {
             if (_.isEmpty(data)) {
                 throw new AppError('does not find deployment');
             }
-            // eslint-disable-next-line no-param-reassign
             data.label_id += 1;
             return data.save({ transaction: t }).then((d) => {
                 return d.label_id;

src/routes/apps.ts

@@ -217,7 +217,6 @@ appsRouter.get(
                                 .getMetricsbyPackageId(v.get('id'))
                                 .then((metrics) => {
                                     if (metrics) {
-                                        // eslint-disable-next-line no-param-reassign
                                         result[v.get('label')] = {
                                             active: metrics.get('active'),
                                             downloaded: metrics.get('downloaded'),
@@ -899,7 +898,6 @@ appsRouter.get(
                             isCurrentAccount = true;
                         }
 
-                        // eslint-disable-next-line no-param-reassign
                         result[key] = {
                             ...value,
                             isCurrentAccount,

src/routes/index.ts

@@ -50,7 +50,6 @@ indexRouter.get(
                     .chosenMan(rs.packageId, rs.rollout, clientUniqueId)
                     .then((data) => {
                         if (!data) {
-                            // eslint-disable-next-line no-param-reassign
                             rs.isAvailable = false;
                             return rs;
                         }
@@ -60,9 +59,7 @@ indexRouter.get(
             .then((rs) => {
                 logger.info('updateCheck success');
 
-                // eslint-disable-next-line no-param-reassign
                 delete rs.packageId;
-                // eslint-disable-next-line no-param-reassign
                 delete rs.rollout;
                 res.send({ updateInfo: rs });
             })

src/routes/indexV1.ts

@@ -50,7 +50,6 @@ indexV1Router.get(
                     .chosenMan(rs.packageId, rs.rollout, clientUniqueId)
                     .then((data) => {
                         if (!data) {
-                            // eslint-disable-next-line no-param-reassign
                             rs.isAvailable = false;
                             return rs;
                         }