fresh icon indicating copy to clipboard operation
fresh copied to clipboard

Bug! Should've ignored format string in logs

Open CaledoniaProject opened this issue 9 years ago • 3 comments

Minimal case,

func main () {
    log.Println ("Hello There! %Y %s", "123")
}

And you will see this in fresh,

15:24:52 app         | 2016/05/16 15:24:52 Hello There! %!Y(MISSING) %!s(MISSING) 123

CaledoniaProject avatar May 16 '16 07:05 CaledoniaProject

@CaledoniaProject you are right, but everything is used in the string format for now. feel free to open a PR if you want to fix it :)

gravityblast avatar May 16 '16 09:05 gravityblast

@pilu I made a PR for this (https://github.com/pilu/fresh/pull/54), please consider ^^

elvinchan avatar Jun 22 '16 02:06 elvinchan

:+1:

CraigChilds94 avatar Sep 26 '17 14:09 CraigChilds94