Merge pull request #1163 from alleyinteractive/feature/issue-1162/sup…

…port-php-8-3

Issue-1162: Support PHP 8.3

.deployignore

@@ -20,6 +20,7 @@ vendor
 .eslintrc.json
 .gitignore
 .nvmrc
+.phpunit-watcher.yml
 .phpcs.xml
 .phpcs-cache.json
 .phpunit.result.cache

.distignore

@@ -21,6 +21,7 @@ vendor
 .gitignore
 .nvmrc
 .phpcs.xml
+.phpunit-watcher.yml
 .phpcs-cache.json
 .phpunit.result.cache
 babel.config.json

.github/workflows/phpcs.yml

@@ -15,7 +15,7 @@ jobs:
   phpcs:
     strategy:
       matrix:
-        php: [ 8.2 ]
+        php: [ '8.3' ]
     if: github.event.pull_request.draft == false
     uses: alleyinteractive/.github/.github/workflows/php-coding-standards.yml@main
     with:

.github/workflows/phpunit.yml

@@ -16,8 +16,12 @@ jobs:
     strategy:
       matrix:
         multisite: [true, false]
-        php: ['8.2', '8.1', '8.0']
+        php: ['8.3', '8.2', '8.1', '8.0']
         wordpress: ["latest"]
+        include:
+          - php: '8.0'
+            wordpress: '6.3'
+            multisite: false
     if: github.event.pull_request.draft == false
     uses: alleyinteractive/.github/.github/workflows/php-tests.yml@main
     with:

.phpcs.xml

@@ -5,9 +5,36 @@
   <!-- Include Alley Rules -->
   <rule ref="Alley-Interactive" />
 
-  <!-- Check for cross-version support for PHP 7.4 and higher. -->
+  <!--
+  Pass some flags to PHPCS:
+    p flag: Show progress of the run.
+    s flag: Show sniff codes in all reports.
+  -->
+  <arg value="ps" />
+
+  <!-- Whenever possible, cache the scan results and re-use those for unchanged files on the next scan. -->
+  <arg name="cache" value=".phpcs-cache.json" />
+
+  <!-- Strip the filepaths down to the relevant bit. -->
+  <arg name="basepath" value="./" />
+
+  <!-- Check up to 20 files simultaneously. -->
+  <arg name="parallel" value="20" />
+
+  <!-- Set severity to 1 to see everything that isn't effectively turned off. -->
+  <arg name="severity" value="1" />
+
+  <!-- Check for cross-version support for PHP 8.0 and higher. -->
   <config name="testVersion" value="8.0-"/>
 
+  <!-- The version set here matches the minimum version tested. -->
+  <config name="minimum_wp_version" value="6.3" />
+
+  <!-- Exclude a few directories and autogenerated files. -->
+  <exclude-pattern>build/</exclude-pattern>
+  <exclude-pattern>node_modules/</exclude-pattern>
+  <exclude-pattern>vendor/</exclude-pattern>
+
   <!-- Set the text domain for i18n. -->
   <rule ref="WordPress.WP.I18n">
     <properties>
@@ -43,31 +70,4 @@
   <rule ref="WordPress.Files.FileName.InvalidClassFileName">
     <exclude-pattern>tests/</exclude-pattern>
   </rule>
-
-  <!--
-  Pass some flags to PHPCS:
-    p flag: Show progress of the run.
-    s flag: Show sniff codes in all reports.
-  -->
-  <arg value="ps" />
-
-  <!-- Whenever possible, cache the scan results and re-use those for unchanged files on the next scan. -->
-  <arg name="cache" value=".phpcs-cache.json" />
-
-  <!-- Strip the filepaths down to the relevant bit. -->
-  <arg name="basepath" value="./" />
-
-  <!-- Check up to 20 files simultaneously. -->
-  <arg name="parallel" value="20" />
-
-  <!-- Set severity to 1 to see everything that isn't effectively turned off. -->
-  <arg name="severity" value="1" />
-
-  <!-- Exclude a few directories and autogenerated files. -->
-  <exclude-pattern>build/</exclude-pattern>
-  <exclude-pattern>node_modules/</exclude-pattern>
-  <exclude-pattern>vendor/</exclude-pattern>
-
-  <!-- The version set here matches the minimum version in readme.txt. -->
-  <config name="minimum_wp_version" value="6.3" />
 </ruleset>

.phpunit-watcher.yml

@@ -0,0 +1,6 @@
+watch:
+  directories:
+    - tests
+    - admin
+    - includes
+  fileMask: '*.php'

apple-news.php

@@ -51,9 +51,9 @@ function apple_news_date( $format, $timestamp = null, $timezone = null ) {
  */
 function apple_news_activate_wp_plugin() {
 	// Check for PHP version.
-	if ( version_compare( PHP_VERSION, '5.3.6' ) < 0 ) {
+	if ( version_compare( PHP_VERSION, '8.0.0' ) < 0 ) {
 		deactivate_plugins( basename( __FILE__ ) );
-		wp_die( esc_html__( 'This plugin requires at least PHP 5.3.6', 'apple-news' ) );
+		wp_die( esc_html__( 'This plugin requires at least PHP 8.0.0', 'apple-news' ) );
 	}
 }
 
@@ -194,9 +194,5 @@ function apple_news_is_classic_editor_plugin_active() {
 		include_once ABSPATH . 'wp-admin/includes/plugin.php';
 	}
 
-	if ( is_plugin_active( 'classic-editor/classic-editor.php' ) ) {
-		return true;
-	}
-
-	return false;
+	return is_plugin_active( 'classic-editor/classic-editor.php' );
 }

phpunit.xml.dist

@@ -1,17 +1,17 @@
 <phpunit
-	bootstrap="tests/bootstrap.php"
-	backupGlobals="false"
-	colors="true"
-	convertErrorsToExceptions="true"
-	convertNoticesToExceptions="true"
-	convertWarningsToExceptions="true"
-	>
-	<php>
-		<const name="APPLE_NEWS_UNIT_TESTS" value="1" />
-	</php>
-	<testsuites>
-		<testsuite name="Unit Tests">
-			<directory prefix="test-" suffix=".php">./tests/</directory>
-		</testsuite>
-	</testsuites>
+  bootstrap="tests/bootstrap.php"
+  backupGlobals="false"
+  colors="true"
+  convertDeprecationsToExceptions="true"
+  convertErrorsToExceptions="true"
+  convertNoticesToExceptions="true"
+  convertWarningsToExceptions="true">
+  <php>
+    <const name="APPLE_NEWS_UNIT_TESTS" value="1" />
+  </php>
+  <testsuites>
+    <testsuite name="Unit Tests">
+      <directory prefix="test-" suffix=".php">./tests/</directory>
+    </testsuite>
+  </testsuites>
 </phpunit>