Problem/Motivation

Unused variable on BlazyFormatterTest

CommentFileSizeAuthor
#4 blazy-unused_variable_on_BlazyFormatterTest-3026135-4-D8.patch1.39 KBthalles
#2 blazy-unused_variable_on_BlazyFormatterTest-3026135-2-D8.patch1.06 KBthalles

Comments

thalles created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
StatusFileSize
new blazy-unused_variable_on_BlazyFormatterTest-3026135-2-D8.patch1.06 KB

Follow the patch!

gausarts’s picture

gausarts commented
Status: Needs review » Needs work

Thanks!

I think we'd better delete this particular method block prefixed with todoTest... as Blazy 2.x no longer supports VEF/ VEM, and it also doesn't run the tests anyway:

public function todoTestBlazyFile() {
  // .....
}
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
StatusFileSize
new blazy-unused_variable_on_BlazyFormatterTest-3026135-4-D8.patch1.39 KB

Follow the patch!

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs work » Needs review

  • gausarts committed 20962d6 on 8.x-2.x authored by thalles 
    Issue #3026135 by thalles: Unused variable on BlazyFormatterTest
gausarts’s picture

gausarts commented
Category: Feature request » Task
Status: Needs review » Fixed

Committed. Thank you for contribution!

Status: Fixed » Closed (fixed)

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