- Issue created by @nicxvan
- First commit to issue fork.
- Merge request !10549Updated comments to remove references to Drupal CI. → (Closed) created by jvbrian
- 🇺🇸United States yujiman85
I checked this out and all mentions of Drupal CI seem to be gone, I also reran the pipeline to get it to pass. I can't comment on the new wording of the comments since I don't know what it's supposed to become so I'll leave this as Needs Review for now.
- 🇺🇸United States smustgrave
Searched for Drupal CI and all 4 instances have been replaced and the MR appears to have good replacements for each wording.
- First commit to issue fork.
- 🇺🇸United States yujiman85
The suggested change was made and committed, looks good.
- 🇳🇿New Zealand quietone
Thanks for working on this.
Just one more thing, the changes here need to be wrapped at 80 characters to meet the Drupal coding standards, https://www.drupal.org/docs/develop/standards/php/php-coding-standards#line-length.
- 🇮🇳India lavanyatalwar
Fixed the 80 char wrapping error
Kindly review :) -
larowlan →
committed 975aad7e on 11.x
Issue #3493279 by jvbrian, xamount, lavanyatalwar, yujiman85, nicxvan,...
-
larowlan →
committed 975aad7e on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.
