- π―π΅Japan ptmkenny
Patch no longer applies and in 8.x-1.x-dev, the critical change has already been made:
"drush.services.yml": ">=9"
It would still be good to add the test though.
- π΅πΉPortugal jcnventura
Please note that Drush 12 and 13 are VERY different beasts than 10 and 11..
Among others, I think drush.services.yml has been dropped, and the command files for Drush 12 should reside in src/Drush/Commands. Also Autowire should be used from Drush 12 onwards. See https://www.drush.org./12.x/commands/
Whatever has been committed, should probably pay attention to that, and limit usage of drush.services to Drush 9-11.
- Status changed to Needs review
10 months ago 4:05am 4 June 2024 - last update
10 months ago 10 pass - πΊπΈUnited States rlhawk Seattle, Washington, United States
Here's a reroll of the patch to add the test.
- First commit to issue fork.
- Merge request !24Issue #3133528 by rlhawk, heddn, davisben: Add test for Drush commands β (Merged) created by japerry
- last update
10 months ago 10 pass - Status changed to RTBC
10 months ago 10:19pm 25 June 2024 - πΊπΈUnited States japerry KVUO
Added some nits above, but otherwise it passed tests and looks good.
- last update
10 months ago 10 pass -
rlhawk β
committed 74d0d36d on 8.x-1.x authored by
japerry β
Issue #3133528 by rlhawk, heddn, davisben: Add test for Drush commands
-
rlhawk β
committed 74d0d36d on 8.x-1.x authored by
japerry β
- π΅πΉPortugal jcnventura
Weird, the post-commit build failed with the following error:
"ERROR: Uploading artifacts as "archive" to coordinator... 413 Request Entity Too Large id=1953767 responseStatus=413 Request Entity Too Large status=413 token=glcbt-64
FATAL: too large"But only the next minor.. Is there a way to request a retest?
- Status changed to Fixed
9 months ago 12:23am 4 July 2024 - πΊπΈUnited States rlhawk Seattle, Washington, United States
I reran the pipeline and it passed the test that failed.
Automatically closed - issue fixed for 2 weeks with no activity.