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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

swentel’s picture

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

Oh man :)

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

This is ready to fly.

claudiu.cristea’s picture

claudiu.cristea
Primary language Romanian
Location Arad 🇷🇴
CreditAttribution: claudiu.cristea commented

RTBC

Xano’s picture

Xano
he/they
Primary language English
Location Southampton
CreditAttribution: Xano commented

d8_debug_statement.patch queued for re-testing.

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
CreditAttribution: webchick 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.