From aaee92a18a885de4c0ef4182042de1a4f9bfe67f Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:34:16 -0400
Subject: [PATCH 01/20] Update to Node 18 on CircleCI

---
 defaults/install/.circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/.circleci/config.yml b/defaults/install/.circleci/config.yml
index 7b10a859..20ecf63a 100644
--- a/defaults/install/.circleci/config.yml
+++ b/defaults/install/.circleci/config.yml
@@ -20,7 +20,7 @@ jobs:
     environment:
       - PALANTIR_ENVIRONMENT: circleci
       - DRUPAL_ROOT: ${drupal.root}
-      - NODE_VERSION: 8
+      - NODE_VERSION: 18
 
     steps:
       - run:

From 8c5ebb3da4761a6c8094efbd9903192c11e9c6e7 Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:36:26 -0400
Subject: [PATCH 02/20] Update deploy-pantheon.example circleci config to use
 node 18

---
 defaults/install/.circleci/deploy-pantheon.example | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/.circleci/deploy-pantheon.example b/defaults/install/.circleci/deploy-pantheon.example
index 544f0073..2d68fbeb 100644
--- a/defaults/install/.circleci/deploy-pantheon.example
+++ b/defaults/install/.circleci/deploy-pantheon.example
@@ -26,7 +26,7 @@
     docker:
       - image: cimg/php:8.1-browsers
     environment:
-      - NODE_VERSION: 8
+      - NODE_VERSION: 18
       - GITHUB_PUBKEY: "github.com,192.30.253.113 ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ=="
       - PANTHEON_PUBKEY: "[codeserver.dev.56f20816-b4b7-4517-a90f-667b4785708d.drush.in]:2222,[35.193.151.123]:2222 ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDSY3gnr0DrbqJJSnEFy6jazDmAdBm4Zs/EkWIQa7x31qgSYyYJMz5V+pk62lBf2BN42VtubwO83vW9G+yG2K1RGOvZJaK5GBvBb/Ws2ZPcp/4sNHpPzkdd75e5/Pk8AWA59XUbJcBWmrDrHMbWV1j2zqPPikxbqGeTTjSx4QR18LIRei5OwT6VQnaVnJqPAqFZ+oCbpr0DL96foL3UEY8EWT/6GH2cANEGZO4ppbhdDw4uG6TaI7S0lxWMQEVy+iwjCNH/nanjd73cwoYd90E0OVdgNDr3hVbIuE6sUW6UwlaAwuyOM/xJYPg1y0rF66958pyVJlZ9KD5A0kY3bHg7"
 

From 591c7f046f607f3d2b0070a5440bd72b02c1b11b Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:37:04 -0400
Subject: [PATCH 03/20] Update deploy-acquia-example circleci config to use
 node 18

---
 defaults/install/.circleci/deploy-acquia-example | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/.circleci/deploy-acquia-example b/defaults/install/.circleci/deploy-acquia-example
index aab77b5a..b0da9a82 100644
--- a/defaults/install/.circleci/deploy-acquia-example
+++ b/defaults/install/.circleci/deploy-acquia-example
@@ -23,7 +23,7 @@
     docker:
       - image: cimg/php:8.1-browsers
     environment:
-      - NODE_VERSION: 8
+      - NODE_VERSION: 18
       - GITHUB_PUBKEY: "github.com,192.30.253.113 ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ=="
       - ACQUIA_PUBKEY: "svn-23450.prod.hosting.acquia.com,54.245.120.237 ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAABAQDit0Rj8j59i9Gj2vOg2TXjIbPswIc27PwncapyBLi9R3FtECXq+md1BfykBJjME83m78CtqkTlWwNZu+LyhF+IYlshKo82JXQzV4kWwi5L9H7KpdyFDMDLVCNVYmU4EQY9ngRq1cwbMBYsfWxv4sSlbsvZv2C8U7luNoc4SiBY4sGpgiMJLFOHWRw6YvhOamLTKfxSoJUT8uRbfkamxas1EK21XjvrdUSN0Bf5LmTdvs/9iIaUH4AlYr0SW0tc2IbIuoFVxuvSzZj2GJjmtCcW1poDMQLTGq+EuO4Mk1WyoUSJC9L8CFonKH7++EPHopvIbPMinO4r2zLEot3lL5TH"
 

From a260fca4145ff50e24dc8ec52bb8c4c8d2bdfa81 Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:39:57 -0400
Subject: [PATCH 04/20] use php 8.2 on circleci

---
 .circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 5ac76ad2..d3c25815 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -6,7 +6,7 @@ jobs:
   build:
     working_directory: ~/project
     docker:
-      - image: cimg/php:8.1-browsers
+      - image: cimg/php:8.2-browsers
       - image: cimg/mysql:5.7
         command: --max_allowed_packet=16M
         environment:

From 63fd8e2b0c6f8826598df99b6b9c6dc099c8362d Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:40:49 -0400
Subject: [PATCH 05/20] Update default installed config.yml to use php 8.2

---
 defaults/install/.circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/.circleci/config.yml b/defaults/install/.circleci/config.yml
index 20ecf63a..0f0dc9f9 100644
--- a/defaults/install/.circleci/config.yml
+++ b/defaults/install/.circleci/config.yml
@@ -9,7 +9,7 @@ jobs:
   build:
     working_directory: ~/${projectname}
     docker:
-      - image: cimg/php:8.1-browsers
+      - image: cimg/php:8.2-browsers
       - image: cimg/mysql:5.7
         command: --max_allowed_packet=16M
         environment:

From 0c0039b8d30411f1141cae85b961fca0b88b82b1 Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Fri, 26 May 2023 13:43:25 -0400
Subject: [PATCH 06/20] add phpcs known issue to readme

---
 README.md | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/README.md b/README.md
index e5564a42..55ec6db8 100644
--- a/README.md
+++ b/README.md
@@ -65,6 +65,10 @@ View a list of other available targets with:
 $> vendor/bin/phing -l
 ```
 
+## Known Issues
+
+When using `phpcs` with PHP 8.1.19, there is a memory leak that will cause the process to hang. It will be killed on CircleCI, and jobs will fail. Pin the container to version 8.1.18 to work around this issue.
+
 ## Additional documentation
 
 * [Configuring the-build](docs/configuration.md)

From b9daa26d25714498c925dcc38fe3a94adeddac22 Mon Sep 17 00:00:00 2001
From: Ray Walters <walters@palantir.net>
Date: Tue, 23 Jan 2024 14:40:20 -0500
Subject: [PATCH 07/20] DEV-30 - Move phpmd config to phpmd.xml.

---
 defaults/install/build.xml |  5 +--
 defaults/install/phpmd.xml | 78 ++++++++++++++++++++++++++++++++++++++
 targets/install.xml        | 21 ++++++++++
 3 files changed, 101 insertions(+), 3 deletions(-)
 create mode 100644 defaults/install/phpmd.xml

diff --git a/defaults/install/build.xml b/defaults/install/build.xml
index 192329a9..4f305052 100644
--- a/defaults/install/build.xml
+++ b/defaults/install/build.xml
@@ -121,9 +121,8 @@
         <exec command="vendor/bin/phpcs" logoutput="true" checkreturn="true" />
 
         <!-- Run PHP Mess Detector. -->
-        <property name="phpmd.command" value="vendor/bin/phpmd ${phpmd.directories} ${phpmd.format} ${phpmd.rulesets} --suffixes ${phpmd.suffixes}" />
-        <echo msg="$> ${phpmd.command}" />
-        <exec command="${phpmd.command}" logoutput="true" checkreturn="true" />
+        <echo msg="$> vendor/bin/phpmd" />
+        <exec command="vendor/bin/phpmd ${phpmd.directories} ${phpmd.format} ${phpmd.rulesets} --suffixes ${phpmd.suffixes}" logoutput="true" checkreturn="true" />
 
         <!-- Run Drupal Check. -->
         <foreach list="${drupal-check.directories}" param="drupal-check.dir" target="drupal-check" />
diff --git a/defaults/install/phpmd.xml b/defaults/install/phpmd.xml
new file mode 100644
index 00000000..c1a66cfc
--- /dev/null
+++ b/defaults/install/phpmd.xml
@@ -0,0 +1,78 @@
+<?xml version="1.0"?>
+<ruleset xmlns="http://pmd.sourceforge.net/ruleset/2.0.0" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" name="PMD Ruleset for Drupal" xsi:schemaLocation="http://pmd.sourceforge.net/ruleset/2.0.0 http://pmd.sourceforge.net/ruleset_2_0_0.xsd">
+  <description>
+    A PMD Ruleset for Drupal coding standards.
+  </description>
+
+  <!--
+    Include each rule explicitly so we know what we have.
+    @see https://github.com/phpmd/phpmd/blob/master/src/main/resources/rulesets/
+  -->
+
+  <!-- Clean Code -->
+  <!--
+  These don't align with Drupal standards, so they are excluded.
+  @todo Static calls are hard to test and extend, is there a way to whitelist the ones that are OK?
+  <rule ref="rulesets/cleancode.xml/BooleanArgumentFlag"/>
+  <rule ref="rulesets/cleancode.xml/ElseExpression"/>
+  <rule ref="rulesets/cleancode.xml/StaticAccess"/>
+  -->
+
+  <!-- Code Size -->
+  <rule ref="rulesets/codesize.xml/CyclomaticComplexity"/>
+  <rule ref="rulesets/codesize.xml/NPathComplexity"/>
+  <rule ref="rulesets/codesize.xml/ExcessiveMethodLength"/>
+  <rule ref="rulesets/codesize.xml/ExcessiveClassLength"/>
+  <rule ref="rulesets/codesize.xml/ExcessiveParameterList"/>
+  <rule ref="rulesets/codesize.xml/ExcessivePublicCount"/>
+  <rule ref="rulesets/codesize.xml/TooManyFields"/>
+
+  <!-- Controversial -->
+  <rule ref="rulesets/controversial.xml/Superglobals"/>
+  <!--
+  These checks do not need to be included since PHPCS will check for style.
+  <rule ref="rulesets/controversial.xml/CamelCaseClassName"/>
+  <rule ref="rulesets/controversial.xml/CamelCasePropertyName"/>
+  <rule ref="rulesets/controversial.xml/CamelCaseMethodName"/>
+  <rule ref="rulesets/controversial.xml/CamelCaseParameterName"/>
+  <rule ref="rulesets/controversial.xml/CamelCaseVariableName"/>
+  -->
+
+  <!-- Design -->
+  <rule ref="rulesets/design.xml/ExitExpression"/>
+  <rule ref="rulesets/design.xml/EvalExpression"/>
+  <rule ref="rulesets/design.xml/GotoStatement"/>
+  <rule ref="rulesets/design.xml/NumberOfChildren"/>
+  <rule ref="rulesets/design.xml/DepthOfInheritance"/>
+  <rule ref="rulesets/design.xml/CouplingBetweenObjects"/>
+  <rule ref="rulesets/design.xml/DevelopmentCodeFragment"/>
+
+  <!-- Naming -->
+  <rule ref="rulesets/naming.xml/ShortVariable">
+    <properties>
+      <!-- Allow $id as a variable name. -->
+      <property name="exceptions" description="Comma-separated list of exceptions" value="id"/>
+    </properties>
+  </rule>
+  <rule ref="rulesets/naming.xml/LongVariable">
+    <properties>
+      <!-- Bump variable length to a more reasonable number. -->
+      <property name="maximum" description="The variable length reporting threshold" value="35"/>
+    </properties>
+  </rule>
+  <rule ref="rulesets/naming.xml/ShortMethodName"/>
+  <rule ref="rulesets/naming.xml/ConstructorWithNameAsEnclosingClass"/>
+  <rule ref="rulesets/naming.xml/ConstantNamingConventions"/>
+  <rule ref="rulesets/naming.xml/BooleanGetMethodName"/>
+
+  <!-- Unused Code -->
+  <rule ref="rulesets/unusedcode.xml/UnusedPrivateField"/>
+  <rule ref="rulesets/unusedcode.xml/UnusedLocalVariable"/>
+  <rule ref="rulesets/unusedcode.xml/UnusedPrivateMethod"/>
+  <!--
+  Hooks often have unused parameters, so ignore this warning.
+  @todo is there a way to allow unused parameters in hooks but not elsewhere?
+  <rule ref="rulesets/unusedcode.xml/UnusedFormalParameter"/>
+  -->
+
+</ruleset>
diff --git a/targets/install.xml b/targets/install.xml
index 4d342313..76c4901f 100644
--- a/targets/install.xml
+++ b/targets/install.xml
@@ -224,6 +224,27 @@
             </filterchain>
         </copy>
 
+        <!-- Copy the phpmd.xml file template.
+             -->
+        <echo>Updating phpmd.xml file for project root.</echo>
+        <if>
+            <equals arg1="${build.host}" arg2="acquia"/>
+            <then>
+                <property name="phpmd_dir" value="docroot"/>
+            </then>
+            <else>
+                <property name="phpmd_dir" value="web"/>
+            </else>
+        </if>
+        <copy file="${phing.dir.install}/../defaults/install/phpmd.xml" tofile="${application.startdir}/phpmd.xml" overwrite="true">
+            <filterchain>
+                <replacetokens>
+                    <token key="projectname" value="${projectname}" />
+                    <token key="webroot" value="${phpmd_dir}" />
+                </replacetokens>
+            </filterchain>
+        </copy>
+
          <!-- Copy other templates into place.
              These copy commands use <expandproperties /> for property substitution.
              -->

From fbd3f0b99adcd9d0f1dc6f34f834fa75e9ab27c9 Mon Sep 17 00:00:00 2001
From: Ray Walters <walters@palantir.net>
Date: Wed, 20 Mar 2024 16:17:00 -0400
Subject: [PATCH 08/20] DEV-30 - Update the path to the phpmd.xml file in
 defaults.

---
 defaults.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults.yml b/defaults.yml
index b56e2fdc..f431b8bc 100644
--- a/defaults.yml
+++ b/defaults.yml
@@ -259,7 +259,7 @@ phpmd:
   # Output format for phpmd.
   format: text
   # Comma-separated list of ruleset paths or names.
-  rulesets: "${build.thebuild.dir}/defaults/standard/phpmd.xml"
+  rulesets: "${build.thebuild.dir}/defaults/install/phpmd.xml"
   # File extensions to review.
   suffixes: php,inc,module,theme,profile,install,test
 

From b19a59e68a3c5927ea46148696169538f4b4eabc Mon Sep 17 00:00:00 2001
From: Ray Walters <walters@palantir.net>
Date: Wed, 20 Mar 2024 16:21:33 -0400
Subject: [PATCH 09/20] DEV-30 - Update path to phpmd rulesets to root
 directory.

---
 defaults.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults.yml b/defaults.yml
index f431b8bc..70ec09b1 100644
--- a/defaults.yml
+++ b/defaults.yml
@@ -259,7 +259,7 @@ phpmd:
   # Output format for phpmd.
   format: text
   # Comma-separated list of ruleset paths or names.
-  rulesets: "${build.thebuild.dir}/defaults/install/phpmd.xml"
+  rulesets: "${build.dir}/phpmd.xml"
   # File extensions to review.
   suffixes: php,inc,module,theme,profile,install,test
 

From d4a2332b3c60286850915348153cbe19b13f9295 Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Wed, 29 May 2024 16:16:18 -0400
Subject: [PATCH 10/20] remove PHP 8.1 known issue from readme

---
 README.md | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/README.md b/README.md
index 55ec6db8..e5564a42 100644
--- a/README.md
+++ b/README.md
@@ -65,10 +65,6 @@ View a list of other available targets with:
 $> vendor/bin/phing -l
 ```
 
-## Known Issues
-
-When using `phpcs` with PHP 8.1.19, there is a memory leak that will cause the process to hang. It will be killed on CircleCI, and jobs will fail. Pin the container to version 8.1.18 to work around this issue.
-
 ## Additional documentation
 
 * [Configuring the-build](docs/configuration.md)

From 78f2c2d93998b42e71564c6318a98130b304cd66 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Fri, 9 Aug 2024 15:38:03 -0400
Subject: [PATCH 11/20] use php 8.3 on circleci

---
 .circleci/config.yml                               | 2 +-
 defaults/install/.circleci/config.yml              | 2 +-
 defaults/install/.circleci/deploy-acquia-example   | 2 +-
 defaults/install/.circleci/deploy-pantheon.example | 2 +-
 4 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 07f8b438..2dcee84a 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -6,7 +6,7 @@ jobs:
   build:
     working_directory: ~/project
     docker:
-      - image: cimg/php:8.2-browsers
+      - image: cimg/php:8.3-browsers
       - image: cimg/mysql:5.7
         command: --max_allowed_packet=16M
         environment:
diff --git a/defaults/install/.circleci/config.yml b/defaults/install/.circleci/config.yml
index 06c7794d..9d7ffc57 100644
--- a/defaults/install/.circleci/config.yml
+++ b/defaults/install/.circleci/config.yml
@@ -9,7 +9,7 @@ jobs:
   build:
     working_directory: ~/${projectname}
     docker:
-      - image: cimg/php:8.2-browsers
+      - image: cimg/php:8.3-browsers
       - image: cimg/mysql:5.7
         command: --max_allowed_packet=16M
         environment:
diff --git a/defaults/install/.circleci/deploy-acquia-example b/defaults/install/.circleci/deploy-acquia-example
index b0da9a82..0ac6a61f 100644
--- a/defaults/install/.circleci/deploy-acquia-example
+++ b/defaults/install/.circleci/deploy-acquia-example
@@ -21,7 +21,7 @@
   deploy:
     working_directory: ~/project
     docker:
-      - image: cimg/php:8.1-browsers
+      - image: cimg/php:8.3-browsers
     environment:
       - NODE_VERSION: 18
       - GITHUB_PUBKEY: "github.com,192.30.253.113 ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ=="
diff --git a/defaults/install/.circleci/deploy-pantheon.example b/defaults/install/.circleci/deploy-pantheon.example
index 2d68fbeb..72f1598f 100644
--- a/defaults/install/.circleci/deploy-pantheon.example
+++ b/defaults/install/.circleci/deploy-pantheon.example
@@ -24,7 +24,7 @@
   deploy:
     working_directory: ~/project
     docker:
-      - image: cimg/php:8.1-browsers
+      - image: cimg/php:8.3-browsers
     environment:
       - NODE_VERSION: 18
       - GITHUB_PUBKEY: "github.com,192.30.253.113 ssh-rsa AAAAB3NzaC1yc2EAAAABIwAAAQEAq2A7hRGmdnm9tUDbO9IDSwBK6TbQa+PXYPCPy6rbTrTtw7PHkccKrpp0yVhp5HdEIcKr6pLlVDBfOLX9QUsyCOV0wzfjIJNlGEYsdlLJizHhbn2mUjvSAHQqZETYP81eFzLQNnPHt4EVVUh7VfDESU84KezmD5QlWpXLmvU31/yMf+Se8xhHTvKSCZIFImWwoG6mbUoWf9nzpIoaSjB+weqqUUmpaaasXVal72J+UX2B+2RPW3RcT0eOzQgqlJL3RKrTJvdsjE3JEAvGq3lGHSZXy28G3skua2SmVi/w4yCE6gbODqnTWlg7+wC604ydGXA8VJiS5ap43JXiUFFAaQ=="

From 5354e48f995a5445a96495261e91521a6b879eab Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Fri, 9 Aug 2024 15:38:17 -0400
Subject: [PATCH 12/20] use php 8.3 on platform.sh

---
 defaults/install/.platform.app.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/.platform.app.yaml b/defaults/install/.platform.app.yaml
index 85c86f91..cc15d342 100644
--- a/defaults/install/.platform.app.yaml
+++ b/defaults/install/.platform.app.yaml
@@ -3,7 +3,7 @@
 # - https://github.com/platformsh/platformsh-example-drupal8/blob/master/.platform.app.yaml
 
 name: 'app'
-type: 'php:7.2'
+type: 'php:8.3'
 
 relationships:
     database: 'mysqldb:mysql'

From 3035b9dbf78ac42b6973b1fd678a65caa3482c62 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Thu, 22 Aug 2024 09:43:44 -0400
Subject: [PATCH 13/20] remove core node package installation step

---
 defaults/install/build.xml | 11 -----------
 1 file changed, 11 deletions(-)

diff --git a/defaults/install/build.xml b/defaults/install/build.xml
index 4f305052..a13abe3a 100644
--- a/defaults/install/build.xml
+++ b/defaults/install/build.xml
@@ -173,17 +173,6 @@
                 <exec dir="${drupal.root}/.." command="yarn install" logoutput="true" />
             </then>
         </if>
-        <echo msg="Seeing if Drupal core npm packages are installed and at the expected version." />
-        <exec dir="${drupal.root}/core" command="yarn install --check-files --verbose" returnProperty="core_dependencies"  logoutput="true" />
-        <if>
-            <not>
-                <equals arg1="${core_dependencies}" arg2="0" />
-            </not>
-            <then>
-                <echo msg="Install Core node dependencies to get eslint libraries." />
-                <exec dir="${drupal.root}/core" command="yarn install" logoutput="true" checkreturn="true" />
-            </then>
-        </if>
     </target>
 
 

From 23aa0f5c41635ec2366dc67ea2f81d158262fde2 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Thu, 22 Aug 2024 09:46:05 -0400
Subject: [PATCH 14/20] remove explicit yarn package installation check since
 yarn already does this

---
 defaults/install/build.xml | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/defaults/install/build.xml b/defaults/install/build.xml
index a13abe3a..bef28a27 100644
--- a/defaults/install/build.xml
+++ b/defaults/install/build.xml
@@ -162,17 +162,8 @@
 
     <!-- Target: install-node-dependencies -->
     <target name="install-node-dependencies" depends="" description="Install project, core, and custom module node dependencies">
-        <echo msg="Seeing if project npm packages are installed and at the expected version." />
-        <exec command="yarn install --check-files --verbose" returnProperty="project_dependencies"  logoutput="true" />
-        <if>
-            <not>
-                <equals arg1="${project_dependencies}" arg2="0" />
-            </not>
-            <then>
-                <echo msg="Install project node dependencies to get eslint libraries." />
-                <exec dir="${drupal.root}/.." command="yarn install" logoutput="true" />
-            </then>
-        </if>
+        <echo msg="Install project node dependencies to get eslint libraries." />
+        <exec dir="${drupal.root}/.." command="yarn install" logoutput="true" />
     </target>
 
 

From f938f89971fb793a10d736841f8fda7040a85787 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Thu, 22 Aug 2024 16:36:58 -0400
Subject: [PATCH 15/20] test this branch with its skeleton counterpart

---
 .circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 2dcee84a..6102eaa7 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -72,7 +72,7 @@ jobs:
       # Create a new project using the drupal-skeleton project
       - run:
           name: Create a new Drupal project
-          command: composer create-project palantirnet/drupal-skeleton example dev-develop --no-interaction
+          command: composer create-project palantirnet/drupal-skeleton example dev-DEV-70-update-install-node-dependencies-task-in-the-build --no-interaction
           working_directory: ~/
 
       # Use this copy of the-build

From 1efbb57549b4c42da4b818ee92031390033b0f33 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Fri, 23 Aug 2024 09:44:50 -0400
Subject: [PATCH 16/20] remove --resolve-plugins-relative-to drupal core option

---
 defaults/install/build.xml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/defaults/install/build.xml b/defaults/install/build.xml
index bef28a27..cdb6ca2c 100644
--- a/defaults/install/build.xml
+++ b/defaults/install/build.xml
@@ -149,7 +149,7 @@
     <target name="eslint" depends="install-node-dependencies">
         <!-- Lint custom module and theme (by default) javascript. -->
         <echo msg="$> yarn run eslint --ext ${eslint.extensions} ${eslint.directory}" />
-        <exec command="yarn run eslint --fix-dry-run --ext ${eslint.extensions} --resolve-plugins-relative-to ${drupal.root}/core --no-error-on-unmatched-pattern ${eslint.directory}" logoutput="true" checkreturn="true" />
+        <exec command="yarn run eslint --fix-dry-run --ext ${eslint.extensions} --no-error-on-unmatched-pattern ${eslint.directory}" logoutput="true" checkreturn="true" />
     </target>
 
 

From 29902fd7d839b22f4425d7ff171004806c23d4a5 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Fri, 23 Aug 2024 13:04:23 -0400
Subject: [PATCH 17/20] revert change to circle config composer create-project
 step

---
 .circleci/config.yml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.circleci/config.yml b/.circleci/config.yml
index 6102eaa7..2dcee84a 100644
--- a/.circleci/config.yml
+++ b/.circleci/config.yml
@@ -72,7 +72,7 @@ jobs:
       # Create a new project using the drupal-skeleton project
       - run:
           name: Create a new Drupal project
-          command: composer create-project palantirnet/drupal-skeleton example dev-DEV-70-update-install-node-dependencies-task-in-the-build --no-interaction
+          command: composer create-project palantirnet/drupal-skeleton example dev-develop --no-interaction
           working_directory: ~/
 
       # Use this copy of the-build

From 240de672f70997b79adf4f405e14d1088b521102 Mon Sep 17 00:00:00 2001
From: Byron Duvall <duvall@palantir.net>
Date: Fri, 23 Aug 2024 16:27:15 -0400
Subject: [PATCH 18/20] uninstall the standard profile instead of switching it
 during first drupal install

---
 targets/drupal.xml | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/targets/drupal.xml b/targets/drupal.xml
index b69405ce..8f21df7d 100644
--- a/targets/drupal.xml
+++ b/targets/drupal.xml
@@ -296,12 +296,9 @@ Or, you can specify the export file directly:
             <param>the_build_utility</param>
         </drush>
 
-        <phingcall target="drupal-change-profile">
-            <property name="new_profile" value="minimal" />
-        </phingcall>
-
         <drush command="pm-uninstall" assume="yes">
             <param>the_build_utility</param>
+            <param>standard</param>
         </drush>
 
         <drush command="config-export" assume="yes" />

From fa0906c36b47e89ba96fe18fdf0412b0f13c71d5 Mon Sep 17 00:00:00 2001
From: Bec White <white@palantir.net>
Date: Mon, 26 Aug 2024 14:18:34 -0500
Subject: [PATCH 19/20] Update default CircleCI config to cache packages used
 by both composer and yarn.

---
 defaults/install/.circleci/config.yml | 21 +++++++++++++--------
 1 file changed, 13 insertions(+), 8 deletions(-)

diff --git a/defaults/install/.circleci/config.yml b/defaults/install/.circleci/config.yml
index 9d7ffc57..64267d6f 100644
--- a/defaults/install/.circleci/config.yml
+++ b/defaults/install/.circleci/config.yml
@@ -50,10 +50,11 @@ jobs:
           name: Configure URL in /etc/hosts
           command: echo 127.0.0.1 ${CIRCLE_PROJECT_REPONAME}.local | sudo tee -a /etc/hosts
 
-      # Composer package cache
+      # Package cache
       - restore_cache:
           keys:
-            - composer-v1-
+            - package-cache-v1-
+
       # Source cache
       - restore_cache:
           keys:
@@ -64,15 +65,19 @@ jobs:
       - run:
           name: Composer install
           command: composer install --no-interaction --prefer-dist
+      - run:
+          name: Yarn install
+          command: yarn install
 
-      # Composer package cache - update when the contents of the Composer cache directory
-      # change
-      - run: ls -1R ~/.cache/composer/ > /tmp/composer-cache.txt
+      # Package cache. This is updated when the contents of the ~/.cache directory change.
+      # Both Composer and Yarn use this directory.
+      - run: ls -1R ~/.cache/ > /tmp/package-cache.txt
       - save_cache:
-          key: composer-v1-{{ checksum "/tmp/composer-cache.txt" }}
+          key: package-cache-v1-{{ checksum "/tmp/package-cache.txt" }}
           paths:
-              - ~/.cache/composer
-      # Source cache - update when branch changes
+              - ~/.cache
+
+      # Source cache. This is updated for each different branch.
       - save_cache:
           key: source-v1-{{ .Branch }}
           paths:

From 7f3b92c78d83fdc03e75740d426d8bf1899151db Mon Sep 17 00:00:00 2001
From: byrond <duvall@palantir.net>
Date: Wed, 28 Aug 2024 16:25:16 -0400
Subject: [PATCH 20/20] remove drupal-change-profile Phing target

---
 targets/drupal.xml | 43 -------------------------------------------
 1 file changed, 43 deletions(-)

diff --git a/targets/drupal.xml b/targets/drupal.xml
index 8f21df7d..aef4d1b6 100644
--- a/targets/drupal.xml
+++ b/targets/drupal.xml
@@ -315,49 +315,6 @@ Or, you can specify the export file directly:
     </target>
 
 
-    <target name="drupal-change-profile" description="Change the install profile on an existing Drupal site.">
-        <fail unless="new_profile" />
-
-        <!-- Get the name of the current profile. -->
-        <drush command="config-get" returnProperty="current_profile">
-            <param>core.extension</param>
-            <param>profile</param>
-            <option name="format">csv</option>
-        </drush>
-
-        <!-- If the current profile is enabled in the core extensions list, disable it. -->
-        <drush command="config-get" returnProperty="current_profile_weight">
-            <param>core.extension</param>
-            <param>module.${current_profile}</param>
-            <option name="format">csv</option>
-        </drush>
-
-        <if>
-            <not><equals arg1="${current_profile_weight}" arg2="" /></not>
-            <then>
-                <drush command="config-delete" assume="yes">
-                    <param>core.extension</param>
-                    <param>module.${current_profile}</param>
-                </drush>
-            </then>
-        </if>
-
-        <!-- Enable the new profile. -->
-        <drush command="config-set" assume="yes">
-            <param>core.extension</param>
-            <param>module.${new_profile}</param>
-            <param>1000</param>
-        </drush>
-
-        <!-- Update the profile setting itself. -->
-        <drush command="config-set" assume="yes">
-            <param>core.extension</param>
-            <param>profile</param>
-            <param>${new_profile}</param>
-        </drush>
-    </target>
-
-
     <target name="drupal-add-multisite" description="Set up a new Drupal multisite.">
         <if>
             <equals arg1="${build.host}" arg2="pantheon" />