Stumbled over a debug statement in our code-base. Patch to remove it is attached.

CommentFileSizeAuthor
d8_debug_statement.patch665 bytesfago
d8_debug_statement.patch665 bytesfago

Comments

swentel’s picture

swentel commented
Status: Needs review » Reviewed & tested by the community

Oh man :)

dawehner’s picture

dawehner
Primary language German
commented

This is ready to fly.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
commented

RTBC

xano’s picture

xano
he/they
Primary language English
Location Southampton
commented

d8_debug_statement.patch queued for re-testing.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Oops. :P Good catch.

Can we get a follow-up to have testbot fail tests if there are debug statements in them, if one already doesn't exist? It's silly that this would sneak in without humans catching them.

Committed and pushed to 8.x. Thanks!

Status: Fixed » Closed (fixed)

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