Merge pull request #12 from ernilambar/quality

Code quality

.github/workflows/code-quality.yml

@@ -0,0 +1,12 @@
+name: Code Quality Checks
+
+on:
+  pull_request:
+  push:
+    branches:
+      - main
+      - master
+
+jobs:
+  code-quality:
+    uses: wp-cli/.github/.github/workflows/reusable-code-quality.yml@main

.github/workflows/regenerate-readme.yml

@@ -0,0 +1,15 @@
+name: Regenerate README file
+
+on:
+  workflow_dispatch:
+  push:
+    branches:
+      - main
+      - master
+    paths-ignore:
+      - "features/**"
+      - "README.md"
+
+jobs:
+  regenerate-readme:
+    uses: wp-cli/.github/.github/workflows/reusable-regenerate-readme.yml@main

features/database-command-basic.feature

@@ -25,7 +25,7 @@ Feature: Basic tests
     Then STDERR should contain:
       """
       Error: Parameter errors:
-       missing --author parameter (Administrator user you want to keep after reset)
+       missing --author parameter (Administrator user you want to keep after reset.)
       """
 
   Scenario: Test author parameter is passed but non-existent user

phpcs.xml.dist

@@ -21,4 +21,13 @@
 	<!-- Rules: Include the base ruleset for WP-CLI projects. -->
 	<rule ref="WP_CLI_CS"/>
 
+	<rule ref="WordPress.NamingConventions.PrefixAllGlobals">
+		<properties>
+			<property name="prefixes" type="array">
+				<element value="Nilambar\WP_CLI_Database"/><!-- Namespaces. -->
+				<element value="wpcli_database_command"/><!-- Global variables and such. -->
+			</property>
+		</properties>
+	</rule>
+
 </ruleset>

src/DatabaseCommand.php

@@ -13,11 +13,11 @@ class DatabaseCommand extends WP_CLI_Command {
 	* ## OPTIONS
 	*
 	 * --author=<username>
-	 * : Administrator user you want to keep after reset
+	 * : Administrator user you want to keep after reset.
 	*
 	 * ## EXAMPLES
 	 *
-	 *     # Reset database and keep `admin` user
+	 *     # Reset database and keep `admin` user.
 	 *     $ wp database reset --author=admin
 	 *
 	 * @when after_wp_load
@@ -62,7 +62,7 @@ public function reset( $args, $assoc_args ) {
 	private function reset_callback( $user ) {
 		WP_CLI::log( 'Resetting...' );
 
-		// We dont want email notification.
+		// We don't want email notification.
 		if ( ! function_exists( 'wp_new_blog_notification' ) ) {
 			function wp_new_blog_notification() {
 				// Silence is golden.
@@ -79,10 +79,10 @@ function wp_new_blog_notification() {
 
 		$prefix = str_replace( '_', '\_', $wpdb->prefix );
 
-		$tables = $wpdb->get_col( "SHOW TABLES LIKE '{$prefix}%'" );
+		$tables = $wpdb->get_col( "SHOW TABLES LIKE '{$prefix}%'" ); // phpcs:ignore WordPress.DB.PreparedSQL
 
 		foreach ( $tables as $table ) {
-			$wpdb->query( "DROP TABLE $table" );
+			$wpdb->query( "DROP TABLE $table" ); // phpcs:ignore WordPress.DB.PreparedSQL
 		}
 
 		// Set site URL.