Merge pull request #164 from MITLibraries/fix-phpcs

Upgrade PHPCS

.github/workflows/ci.yml

@@ -8,7 +8,7 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        php-version: ['8.1', '8.2']
+        php-version: ['8.1', '8.2', '8.3']
 
     steps:
     - uses: actions/checkout@v2

.gitignore

@@ -9,6 +9,7 @@
 # but in most cases, we don't want to version control anything in here.
 # It's unlikely that you would need to change any of these rules.
 web/app/upgrade
+web/app/uploads
 web/app/uploads/*
 !web/app/uploads/.gitkeep
 web/app/cache/*

composer.json

@@ -129,10 +129,8 @@
     "wpengine/advanced-custom-fields-pro": "^5"
   },
   "require-dev": {
-    "dealerdirect/phpcodesniffer-composer-installer": "^0.7.2",
     "roave/security-advisories": "dev-latest",
-    "squizlabs/php_codesniffer": "^3.6.2",
-    "wp-coding-standards/wpcs": "^2.3"
+    "wp-coding-standards/wpcs": "^3.0.0"
   },
   "config": {
     "optimize-autoloader": true,
@@ -141,9 +139,9 @@
     "allow-plugins": {
       "composer/installers": true,
       "cweagans/composer-patches": true,
-      "dealerdirect/phpcodesniffer-composer-installer": true,
       "pivvenit/acf-pro-installer": true,
-      "roots/wordpress-core-installer": true
+      "roots/wordpress-core-installer": true,
+      "dealerdirect/phpcodesniffer-composer-installer": true
     },
     "sort-packages": true
   },
@@ -218,15 +216,15 @@
     "security": [
       "phpcs --standard=phpcs.security.xml"
     ],
+    "lint": [
+      "phpcs"
+    ],
     "syntax": [
       "find . -path ./libs -prune -o \\( -name '*.php' \\) -exec php -lf {} \\;"
     ],
     "install-sage": [
       "bash ./private/scripts/sage-theme-install.sh"
     ],
-    "test": [
-      "phpcs"
-    ],
     "upstream-require": [
       "WordPressComposerManaged\\ComposerScripts::upstreamRequire"
     ]