Merge pull request #131 from wp-graphql/feat/129-replace-updater-in-build

feat: replace Appsero Updater.php with blank class to pass .org req's

Author: jasonbahl

build/Updater.php

@@ -0,0 +1,14 @@
+<?php
+namespace Appsero;
+
+/**
+ * Temp replacement for the Updater.php class
+ * to allow the plugin to pass WordPress.org
+ * review process until Appsero has a formal solution.
+ *
+ * This file is used to replace the Updater.php class in the Appsero vendor dep after
+ * composer install is run, before the plugin is bundled as a .zip
+ *
+ * see: https://github.com/Appsero/client/issues/34
+ */
+class Updater {}

composer.json

@@ -73,7 +73,11 @@
     ],
     "build-app": "@docker-build -a",
     "build-test": "@docker-build -t",
-    "build-plugin": "composer install --no-dev && composer run-script zip && composer install",
+    "build-plugin": [
+      "composer install --no-dev",
+      "composer run-script zip",
+      "composer install"
+    ],
     "run-app": "@docker-run -a",
     "run-test": "@docker-run -t",
     "lint": "vendor/bin/phpcs",
@@ -89,6 +93,10 @@
     "phpstan": [
       "phpstan analyze --ansi --memory-limit=1G"
     ],
+    "post-install-cmd": [
+      "rm ./vendor/appsero/client/src/Updater.php",
+      "cp ./build/Updater.php ./vendor/appsero/client/src/"
+    ],
     "zip": [
       "mkdir -p plugin-build/wpgraphql-acf",
       "rsync -rc --exclude-from=.distignore --exclude=plugin-build . plugin-build/wpgraphql-acf/ --delete --delete-excluded -v",