Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread Andreas Götz
Forgive me. I have overlooked the fact, that this is a generated file that gets regenerated on the CI. Only in this case it must not since the generator does not cover a specific case and needs manual touch-up, hence must not be modified afterwrds. Sorry for the noise and wish you a nice Sunday

Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread Jan Mercl
On Sun, Apr 16, 2023 at 12:51 PM Andreas Götz wrote: > Isn't gofmt ist part of the go command? Go ist the latest version: It is but it's a separate binary/executable file. Go version is one thing, the version of the binary file that is actually executed when '$ gofmt' is invoked is another thing

Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread Andreas Götz
Isn't gofmt ist part of the go command? Go ist the latest version: Setup go version spec ^1.20 Found in cache @ /

Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread 'Axel Wagner' via golang-nuts
I can't tell what version of gofmt is used? It does not seem to be manually installed, so I assume it is the one that comes with the image. Is it possible that's too old? I don't know why that would matter here, though, the code doesn't seem to be using any new language features. On Sun, Apr 16, 2

Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread Andreas Götz
The entire file is https://raw.githubusercontent.com/evcc-io/evcc/a5f7bb0100c4427a2f611f82b742f56a449681de/charger/keba-modbus_decorators.go, part of https://github.com/evcc-io/evcc/pull/7485. Seems entirely find to me and works locally which makes it so confusing. Also formats fine: https://

Re: [go-nuts] gofmt: expected declaration, found ')'

2023-04-16 Thread Jan Mercl
On Sun, Apr 16, 2023 at 12:17 PM Andreas Götz wrote: > > Good morning. Please excuse me for asking here- I'm stuck with something that > looks like a gofmt bug though that seems very unlikely. > > In CI > (https://github.com/evcc-io/evcc/actions/runs/4712710735/jobs/8357929411?pr=7485) > I'm us