Just noticed a very very trivial thing while reading the code.... a missing space ... might as well fix it in public :)

CommentFileSizeAuthor
space-0.patch548 bytesmartin107
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

martin107 created an issue. See original summary.

klausi’s picture

klausi
he/him||they/them
Primary language German
Location 🇦🇹 Vienna
CreditAttribution: klausi commented
Status: Needs review » Needs work 
+++ b/tests/src/Unit/RulesActionBaseTest.php
@@ -17,7 +17,7 @@ use Drupal\rules\Core\RulesActionBase;
+   * Tests that a missing label throw a an exception.

should be "throws an".

And can you file a pull request instead of patches at https://github.com/fago/rules ?

martin107’s picture

martin107 CreditAttribution: martin107 commented
Status: Needs work » Fixed

There is a good pull request here

https://github.com/fago/rules/pull/277

Thanks visabhishek

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.