Problem/Motivation

The opening comment
/**
* @file
* Contains \Drupal\Tests\editor\Kernel\EditorFilterIntegration.
*/
does not show correct class name.

Proposed resolution

Change the comment to match the class

/**
* @file
* Contains \Drupal\Tests\editor\Kernel\EditorFilterIntegrationTest.
*/

CommentFileSizeAuthor
#3 fix_comment_of-2585655-3.patch548 bytesThoughtForm

Comments

ThoughtForm created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Issue tags: +Novice

@ThoughtForm is working on this as part of the PNWDS.

ThoughtForm’s picture

ThoughtForm commented
Status: Active » Needs review
StatusFileSize
new fix_comment_of-2585655-3.patch548 bytes
joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Assigned: ThoughtForm » Unassigned
Status: Needs review » Reviewed & tested by the community

This looks perfect the "Contains" now matches the namespaces and class. Thank you @ThoughtForm for this fix.

mile23’s picture

mile23
Primary language English
Location Seattle, WA
commented

ThoughtForm++

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Priority: Normal » Minor
Issue tags: +Quickfix, +Documentation

Thanks!

Signaling to committers this is super low risk.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Issue tags: +rc eligible
alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Fixed

Yep these types of non code cleanups are rc eligible. Committed 77a7014 and pushed to 8.0.x. Thanks!

  • alexpott committed 77a7014 on 8.0.x 
    Issue #2585655 by ThoughtForm: Fix comment of namespace and class \...

Status: Fixed » Closed (fixed)

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