unused property

Author: Kaylie Kwon

src/cli/commands/install.js

@@ -233,8 +233,8 @@ export class Install {
       const projectManifestJson = await this.config.readJson(loc);
       await normalizeManifest(projectManifestJson, this.config.lockfileFolder, this.config, true);
 
-      this._resolutions.init(projectManifestJson.resolutions);
       Object.assign(this.resolutions, projectManifestJson.resolutions);
+      this._resolutions.init(this.resolutions);
       Object.assign(manifest, projectManifestJson);
 
       const pushDeps = (depType, manifest: Object, {hint, optional}, isUsed) => {

src/resolutions.js

@@ -29,25 +29,20 @@ export type ResolutionEntry = {
 
 export default class Resolutions {
   constructor(config: Config) {
-    this._resolutions = map();
     this.resolutionsByPackage = map();
     this.resolvedPatterns = map();
     this.config = config;
     this.reporter = config.reporter;
   }
 
-  _resolutions: ResolutionEntry;
   resolutionsByPackage: ResolutionMap;
   resolvedPatterns: ResolvedPatternsMap;
   config: Config;
   reporter: Reporter;
 
-  init(resolutions: ResolutionEntry) {
-    this._resolutions = {...this._resolutions, ...resolutions};
-    this.resolvedPatterns = map();
-
-    Object.keys(this._resolutions).map(pattern => {
-      const info = this.parsePatternInfo(pattern, this._resolutions[pattern]);
+  init(resolutions: ?ResolutionEntry = {}) {
+    Object.keys(resolutions).map(pattern => {
+      const info = this.parsePatternInfo(pattern, resolutions[pattern]);
 
       if (info) {
         const resolution = this.resolutionsByPackage[info.name] || [];