Fix the YAML syntax in include.drupalci.main.yml

diff --git a/includes/include.drupalci.main.yml b/includes/include.drupalci.main.yml index ed7ee47..55851a9 100644 --- a/includes/include.drupalci.main.yml +++ b/includes/include.drupalci.main.yml @@ -331,9 +331,9 @@ when: never .phpunit-tests-exist-rule: &phpunit-tests-exist-rule - - exists: - - '**/tests/**/*Test.php' - when: on_success + exists: + - '**/tests/**/*Test.php' + when: on_success .nightwatch-tests-exist-rule: &nightwatch-tests-exist-rule - exists:

Comments & Activities

Issue created by @jibran
@jibran opened merge request.
Comment 23 days ago →
🇨🇦Canada jibran Toronto, Canada
https://git.drupalcode.org/project/gitlab_templates/-/merge_requests/390 is ready for review.
Comment 23 days ago →
🇪🇸Spain fjgarlin
Can you give us links of failing pipelines due to this?
Comment 4 days ago →
🇨🇦Canada jibran Toronto, Canada
I tried to reproduce it in Draft: Issue #3540343: No Fix the YAML syntax in include.drupalci.main.yml. and Draft: Issue #3540343: Fix the YAML syntax in include.drupalci.main.yml. But unable to make them fail, so it seems it is a non-issue.
Comment 4 days ago →
🇪🇸Spain fjgarlin
Ok, thanks for confirming it's not an issue.
Comment 4 days ago →
🇬🇧United Kingdom jonathan1055
Yes, I did not think we had any syntax errors, as we would have discovered this by now. However, it is interesting that the changes @jibran made in the MR are also valid yaml and the rules seems to work as intended. The various branches in the downstream pipelines contained PHPUnit and/or Nightwatch jobs correctly.
Comment 4 days ago →
System Message
fjgarlin → closed merge request !390
Comment 4 days ago →
🇬🇧United Kingdom jonathan1055
Also could @jibran or @fjgarlin close MR390 as it is not going to be merged now.

Fix the YAML syntax in include.drupalci.main.yml

Problem/Motivation

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Comments & Activities