When I was glancing through MigrateEvent documentation, I noticed one case where the event mentioned in the docblock didn't match up with the actual class name, and when I copied and pasted from there, it caused my code to fail until I found that I was using the wrong classname/path. So... attaching a patch in the next comment to clean up this documentation :)

CommentFileSizeAuthor
#2 typo_in_documentation-2786389-2.patch601 bytesgeerlingguy

Comments

geerlingguy created an issue. See original summary.

geerlingguy’s picture

geerlingguy commented
Status: Active » Needs review
StatusFileSize
new typo_in_documentation-2786389-2.patch601 bytes
mikeryan’s picture

mikeryan
he/him
Primary language English
Location Pittsfield, MA, USA
commented
Priority: Normal » Minor
Status: Needs review » Reviewed & tested by the community

copypasta--

alexpott’s picture

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

Committed and pushed 6fc9b0d to 8.3.x and e5073be to 8.2.x and b14f925 to 8.1.x. Thanks!

I checked the rest of the class - looks good.

  • alexpott committed 6fc9b0d on 8.3.x 
    Issue #2786389 by geerlingguy: Typo in documentation for...

  • alexpott committed e5073be on 8.2.x 
    Issue #2786389 by geerlingguy: Typo in documentation for...

  • alexpott committed b14f925 on 8.1.x 
    Issue #2786389 by geerlingguy: Typo in documentation for...

Status: Fixed » Closed (fixed)

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