Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Problem/Motivation
Some descriptions Test files are malformed. For example :
/**
* @file
* Contains Drupal\system\Tests\Form\ElementsAccessTest.
*/
Miss a \.
/**
* @file
* Definition of Drupal\config\Tests\ConfigEntityListMultilingualTest.
*/
Use "Definition of" instead of "Contains".
Proposed resolution
- Add this missing \ to all tests definitions
- use "Contains" in all tests definitions.
RC Target evaluation
Only changes comments to make them consistent. Not disruptive at all.
Comment | File | Size | Author |
---|---|---|---|
#4 | bad_file_description-2598840-4.patch | 3.54 KB | GoZ |
Comments
Comment #2
GoZ CreditAttribution: GoZ at Centarro commentedComment #3
GoZ CreditAttribution: GoZ at Centarro commentedComment #4
GoZ CreditAttribution: GoZ at Centarro commentedComment #5
DuaelFrBefore:
After:
Comment:
With that patch applied, 28 missing leading antislash remains outside of the tests. You should open a follow-up issue or extend the scope of this one to fix them.
Comment #6
DuaelFrBetter tagging.
Comment #7
xjmThanks @GoZ and @DuaelFr. Nice small cleanup of a few things that followed an old standard. I also double-checked the fixes locally with
git diff --color-words
.Committed and pushed to 8.0.x. Thanks!