@automattic/calypso-config - npm Package Compare versions

Comparing version 1.1.0 to 1.2.0

__tests__/__mocks__/fs.js

@@ -40,2 +40,6 @@ /**
 			myenvlocal_override: 'myenv',
+			features: {
+				enabledFeature3: true,
+				enabledFeature1: false,
+			}
 		} ),
@@ -42,0 +46,0 @@ 'myenv.local.json': toJSON( {

__tests__/parser.js

@@ -38,6 +38,7 @@ jest.mock( 'fs' );
 		expect( data ).toHaveProperty( 'features', {
-			enabledFeature1: true,
+			enabledFeature1: false,
 			enabledFeature2: true,
 			disabledFeature1: false,
 			disabledFeature2: false,
+			enabledFeature3: true,
 		} );
@@ -44,0 +45,0 @@ } );

jest.config.js

 module.exports = {
+	testURL: 'http://localhost/',
 	modulePaths: [
@@ -3,0 +4,0 @@ '<rootDir>/src/',

package.json

 {
   "name": "@automattic/calypso-config",
-  "version": "1.1.0",
+  "version": "1.2.0",
   "description": "Simple server and client config module. Originally written for wp-calypso.",
@@ -5,0 +5,0 @@ "main": "src/index.js",

src/parser.js

@@ -21,3 +21,2 @@ /**
 		}, defaultOpts ),
-		data = {},
 		configFiles = [
@@ -34,6 +33,14 @@ path.resolve( configPath, '_shared.json' ),
 
-	configFiles.forEach( function( file ) {
-		Object.assign( data, getDataFromFile( file ) );
-	} );
+	const data = configFiles.reduce( ( combinedData, file ) => {
+		const fileData = getDataFromFile( file );
+		const features = Object.assign( {}, combinedData.features );
+		const mergedData = Object.assign( combinedData, fileData );
 
+		if ( fileData.hasOwnProperty( 'features' ) ) {
+			mergedData.features = Object.assign( features, fileData.features );
+		}
+
+		return mergedData;
+	}, {} );
+
 	if ( data.hasOwnProperty( 'features' ) ) {
@@ -40,0 +47,0 @@ enabledFeatures.forEach( function( feature ) {

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

New author

Supply chain risk

A new npm collaborator published a version of the package for the first time. New collaborators are usually benign additions to a project, but do indicate a change to the security surface area of a package.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Lines of code

395

increased by2.86%

Worsened metrics

Total package byte prevSize

17381

decreased by-2.65%

Number of package files

20

decreased by-4.76%

Number of medium supply chain risk alerts

1

increased byInfinity%

No dependency changes