- Issue created by @jahanzaib_at_straker
- 🇳🇿New Zealand jahanzaib_at_straker
testing this patch to address the composer error noticed in the dispatcher.
- 🇺🇸United States npaudyal001
Saw no effect on composure.json changes. Testing the drupalci.yml commands.
-
npaudyal001 →
authored 29678c47 on 3.9.x
Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...
-
npaudyal001 →
authored 29678c47 on 3.9.x
-
npaudyal001 →
authored 6e506ee4 on 3.9.x
Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...
-
npaudyal001 →
authored 6e506ee4 on 3.9.x
-
npaudyal001 →
authored 08f488ca on 3.9.x
Issue #3344857 by npaudyal001, jahanzaib_at_straker: Automated test...
-
npaudyal001 →
authored 08f488ca on 3.9.x
- Status changed to Needs work
over 1 year ago 8:18pm 18 May 2023 - 🇺🇸United States DamienMcKenna NH, USA
Did you mean to commit the changes to the 3.9.x branch and not the 4.0.x branch?
- 🇳🇿New Zealand jeromeroberts
Attempting a patch to update the drupalci build file. - start with 3.9.x as not 100% sure this will work.
During investigation of Nick's patch to set allow-plugin true for cweagans, it was noted that the build was failing before even attempting to set allow-plugin true.
I've found some references that suggest we may be able to add the entire build file with unset default values which will enable us to move Nick's commands further up the build file and thus running within the assemble_codebase step before composer install.
Creating this build file on my local drupalci instance resolved the cweagan error.
I found once the testing commenced, there were some failing tests though - likely unrelated. - last update
over 1 year ago Composer require-dev failure - last update
over 1 year ago Composer require-dev failure - 🇳🇿New Zealand jeromeroberts
Yeah - so cannot patch the drupalci.yml file. Because Jenkins loads this file before the patch is applied and therefor not honoured by the Jenkins test.
To test this change, you'd need to apply the changes directly to repo branch rather than a patch. Rollback/reset the commit if it does not work. -
jeromeroberts →
authored 850f9697 on 3.9.x
Issue #3344857 by npaudyal001, jahanzaib_at_straker, jeromeroberts:...
-
jeromeroberts →
authored 850f9697 on 3.9.x
- 🇳🇿New Zealand jeromeroberts
Trying a different approach to last time.
cweagans is included via the acquia/cohesion package. This change instead attempts to allow-plugin for for cweagans and then install the dev/require-dev package after.
If this works, it would mean acquia/cohesion composer dev dependency will need to be managed through drupalci.yml rather than composer.json.
Just try 3.9.x branch first. - last update
over 1 year ago 207 pass, 321 fail - last update
over 1 year ago Patch Failed to Apply - last update
over 1 year ago 206 pass, 323 fail - 🇺🇸United States DamienMcKenna NH, USA
All current work is going into the 4.0.x branch, shouldn't you focus your efforts there instead of 3.9.
- 🇳🇿New Zealand jeromeroberts
Okay - now we know that moved things along. Here's the same patch changes for the latest 4.0.x branch.
- last update
over 1 year ago 510 pass, 323 fail - last update
over 1 year ago 511 pass, 321 fail -
jeromeroberts →
authored 721fd050 on 4.0.x
Issue #3344857 by npaudyal001, jeromeroberts, jahanzaib_at_straker:...
-
jeromeroberts →
authored 721fd050 on 4.0.x
- 🇺🇸United States npaudyal001
@jeromeroberts, With the patch it seems to build fine, but after the commit tests on other patches are throwing composer config error. I am reverting this commit to test if the error is coming from here.
-
jeromeroberts →
authored 1bf20847 on 4.0.x
Issue #3344857 by npaudyal001, jeromeroberts, jahanzaib_at_straker:...
-
jeromeroberts →
authored 1bf20847 on 4.0.x
- Status changed to Fixed
about 1 year ago 3:54am 16 September 2023 Automatically closed - issue fixed for 2 weeks with no activity.