Per a suggestion by @jessebeach, here is a new song to add to the library, "What Does Testbot Say?"

CommentFileSizeAuthor
#4 what-does-testbot-say-2203045-4.patch2.12 KBRainbowArray
#4 interdiff-2203045-2-4.txt506 bytesRainbowArray
#2 interdiff-what-does-testbot-say-1-2.txt419 bytesRainbowArray
#2 what-does-testbot-say-2.patch2.15 KBRainbowArray
what-does-testbot-say.patch2.15 KBRainbowArray
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

jessebeach’s picture

jessebeach CreditAttribution: jessebeach commented
Status: Needs review » Needs work

AHAHAHahahahaAHAHAHaAAH. Genius. Bravo. I love it.

And it wouldn't be Drupal without a review!

+WHAT THE FOX SAY?

There's one stray reference to FOX in the patch.

Bot says needs work!

RainbowArray’s picture

RainbowArray
she/her
Primary language English
Location Twin Cities, Minnesota
CreditAttribution: RainbowArray commented
Status: Needs work » Needs review
FileSize
what-does-testbot-say-2.patch2.15 KB
interdiff-what-does-testbot-say-1-2.txt419 bytes

Good catch! Here's a revised patch along with an interdiff indicating the change.

yched’s picture

yched CreditAttribution: yched commented

s/TESTBOY/TESTBOT :-)

(unless, like in Snowpiercer, we find out it's an 8-year old boy that runs our tests by hand - which would be an unpleasant revelation)

RainbowArray’s picture

RainbowArray
she/her
Primary language English
Location Twin Cities, Minnesota
CreditAttribution: RainbowArray commented
FileSize
interdiff-2203045-2-4.txt506 bytes
what-does-testbot-say-2203045-4.patch2.12 KB

Fixed this egregious typo on my part. As well as a missing newline.

Hopefully this gets this patch to RTBC.

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Component: all-songs » lyrics and chords

#2100513: Add all songs to a single branch (master)

YesCT’s picture

YesCT CreditAttribution: YesCT commented
Status: Needs review » Fixed

http://drupalcode.org/project/drupal_lyrics.git/commit/9421aed

Status: Fixed » Closed (fixed)

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