yay icon indicating copy to clipboard operation
yay copied to clipboard

fix: added missing spaces

Open Absobel opened this issue 1 year ago • 2 comments

image

There were missing spaces there, so I added them to make it pretty.

Absobel avatar Mar 06 '24 15:03 Absobel

Hey @Absobel is it possible to do this formatting where the errors are output instead?

Jguer avatar Mar 07 '24 08:03 Jguer

I forgot I made this, I just had the problem again. Anyway, would it be better to modify al the calls to Warnln following a call to DependOn ? (it does that only for circular dependencies tho)

if err := graph.DependOn(depName, parentPkgName); err != nil {
g.logger.Warnln(depString, " ", parentPkgName, " ", err)
}

But its strange because the calls to Errorln or things like that don't seem to have this problem. And I don't know how to reliably test that because I would need to find a packages with circular dependencies or smth

Don't hesitate to tell me if I'm missing something totally obvious, I'm not used to make contributions

Absobel avatar Mar 27 '24 12:03 Absobel