Part of meta issue #500866: [META] remove t() from assert message.

CommentFileSizeAuthor
#7 syslog-1787516-7.patch640 bytesdcam
#1 1787516-1-t-syslog.patch1.12 KBlars toomre

Comments

lars toomre’s picture

lars toomre commented
StatusFileSize
new 1787516-1-t-syslog.patch1.12 KB

This is the smallest patch yet for this initiative. Only one change needed!

lars toomre’s picture

lars toomre commented
Status: Active » Needs review

Helps to set for review!

xjm’s picture

xjm
she/her
Primary language English
commented
Assigned: xjm » Unassigned
Status: Needs review » Reviewed & tested by the community

Nice. One assertion!

lars toomre’s picture

lars toomre commented

It was a difficult one to figure out! *smiles*

xjm’s picture

xjm
she/her
Primary language English
commented
Assigned: Unassigned » jhodgdon
jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Patch (to be ported)

Thanks! committed to 8.x.

dcam’s picture

dcam commented
Status: Patch (to be ported) » Needs review
StatusFileSize
new syslog-1787516-7.patch640 bytes

Backported #1 to D7.

lars toomre’s picture

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

This is ready to go. RTBC!

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Assigned: Unassigned » jhodgdon

Tum te tum...

jhodgdon’s picture

jhodgdon
she/her
Primary language English
commented
Assigned: jhodgdon » Unassigned
Status: Reviewed & tested by the community » Fixed

Thanks all! Committed to 7.x (gotta love those one-liners).

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