Flatten modules into top-level directory

WordPress · Mar 24, 2017 · fa8cd1c · fa8cd1c
1 parent 3c35361
commit fa8cd1c
Show file tree

Hide file tree

Showing 20 changed files with 11 additions and 19 deletions.
diff --git a/.eslintignore b/.eslintignore
@@ -1 +1 @@
-modules/*/build
+build
diff --git a/modules/blocks/components/editable/index.js → blocks/components/editable/index.js b/modules/blocks/components/editable/index.js → blocks/components/editable/index.js
diff --git a/modules/blocks/index.js → blocks/index.js b/modules/blocks/index.js → blocks/index.js
diff --git a/modules/blocks/parser/index.js → blocks/parser/index.js b/modules/blocks/parser/index.js → blocks/parser/index.js
diff --git a/modules/blocks/parser/post.pegjs → blocks/parser/post.pegjs b/modules/blocks/parser/post.pegjs → blocks/parser/post.pegjs
diff --git a/modules/blocks/parser/test/index.js → blocks/parser/test/index.js b/modules/blocks/parser/test/index.js → blocks/parser/test/index.js
diff --git a/modules/blocks/test/index.js → blocks/test/index.js b/modules/blocks/test/index.js → blocks/test/index.js
diff --git a/modules/editor/assets/stylesheets/main.scss → editor/assets/stylesheets/main.scss b/modules/editor/assets/stylesheets/main.scss → editor/assets/stylesheets/main.scss
diff --git a/modules/editor/blocks/index.js → editor/blocks/index.js b/modules/editor/blocks/index.js → editor/blocks/index.js
diff --git a/modules/editor/blocks/text-block/index.js → editor/blocks/text-block/index.js b/modules/editor/blocks/text-block/index.js → editor/blocks/text-block/index.js
diff --git a/modules/editor/editor/editor.js → editor/editor/editor.js b/modules/editor/editor/editor.js → editor/editor/editor.js
diff --git a/modules/editor/editor/index.js → editor/editor/index.js b/modules/editor/editor/index.js → editor/editor/index.js
diff --git a/modules/editor/index.js → editor/index.js b/modules/editor/index.js → editor/index.js
diff --git a/modules/editor/inserter/button.js → editor/inserter/button.js b/modules/editor/inserter/button.js → editor/inserter/button.js
diff --git a/modules/editor/inserter/index.js → editor/inserter/index.js b/modules/editor/inserter/index.js → editor/inserter/index.js
diff --git a/modules/editor/inserter/style.scss → editor/inserter/style.scss b/modules/editor/inserter/style.scss → editor/inserter/style.scss
diff --git a/modules/element/index.js → element/index.js b/modules/element/index.js → element/index.js
diff --git a/index.php b/index.php
@@ -40,8 +40,8 @@ function gutenberg_register_scripts() {
 	wp_register_script( 'react-dom', 'https://unpkg.com/react-dom@15/dist/react-dom' . $suffix . '.js', array( 'react' ) );
 
 	// Editor
-	wp_register_script( 'wp-element', plugins_url( 'modules/element/build/index.js', __FILE__ ), array( 'react', 'react-dom' ) );
-	wp_register_script( 'wp-blocks', plugins_url( 'modules/blocks/build/index.js', __FILE__ ), array( 'wp-element' ) );
+	wp_register_script( 'wp-element', plugins_url( 'element/build/index.js', __FILE__ ), array( 'react', 'react-dom' ) );
+	wp_register_script( 'wp-blocks', plugins_url( 'blocks/build/index.js', __FILE__ ), array( 'wp-element' ) );
 }
 add_action( 'init', 'gutenberg_register_scripts' );
 
@@ -58,11 +58,11 @@ function gutenberg_scripts_and_styles( $hook ) {
 	if ( 'toplevel_page_gutenberg' === $hook ) {
 		// Scripts
 		wp_register_script( 'gutenberg-content', plugins_url( 'post-content.js', __FILE__ ) );
-		wp_enqueue_script( 'wp-editor', plugins_url( 'modules/editor/build/index.js', __FILE__ ), array( 'wp-blocks', 'wp-element', 'gutenberg-content' ), false, true );
+		wp_enqueue_script( 'wp-editor', plugins_url( 'editor/build/index.js', __FILE__ ), array( 'wp-blocks', 'wp-element', 'gutenberg-content' ), false, true );
 		wp_add_inline_script( 'wp-editor', 'wp.editor.createEditorInstance( \'editor\', { content: window.content } );' );
 
 		// Styles
-		wp_enqueue_style( 'wp-editor', plugins_url( 'modules/editor/build/style.css', __FILE__ ) );
+		wp_enqueue_style( 'wp-editor', plugins_url( 'editor/build/style.css', __FILE__ ) );
 	}
 }
 

diff --git a/package.json b/package.json
@@ -13,7 +13,7 @@
   "scripts": {
     "test-unit": "cross-env NODE_ENV=test webpack && mocha build --require bootstrap-test.js",
     "build": "cross-env NODE_ENV=production webpack",
-    "lint": "eslint modules",
+    "lint": "eslint editor blocks element",
     "dev": "webpack --watch",
     "test": "npm run lint && npm run test-unit"
   },

diff --git a/webpack.config.js b/webpack.config.js
@@ -3,26 +3,18 @@
  */
 
 const glob = require( 'glob' );
-const path = require( 'path' );
 const webpack = require( 'webpack' );
 const ExtractTextPlugin = require( 'extract-text-webpack-plugin' );
 
-/**
- * Base path from which modules are to be discovered.
- *
- * @type {String}
- */
-const BASE_PATH = './modules';
-
 const config = {
 	entry: {
-		blocks: BASE_PATH + '/blocks/index.js',
-		editor: BASE_PATH + '/editor/index.js',
-		element: BASE_PATH + '/element/index.js'
+		blocks: './blocks/index.js',
+		editor: './editor/index.js',
+		element: './element/index.js'
 	},
 	output: {
 		filename: '[name]/build/index.js',
-		path: path.resolve( BASE_PATH ),
+		path: __dirname,
 		library: [ 'wp', '[name]' ],
 		libraryTarget: 'this'
 	},
@@ -82,7 +74,7 @@ switch ( process.env.NODE_ENV ) {
 
 	case 'test':
 		config.target = 'node';
-		config.entry = glob.sync( BASE_PATH + '/**/test/*.js' );
+		config.entry = glob.sync( `./{${ Object.keys( config.entry ).join() }}/test/*.js` );
 		config.externals = [ require( 'webpack-node-externals' )() ];
 		config.output = {
 			filename: 'build/test.js',