Andrew Farries
Andrew Farries
/hold because it's based on https://github.com/gitpod-io/gitpod/pull/11949
/hold because it's based on https://github.com/gitpod-io/gitpod/pull/11985
/werft run :+1: started the job as [gitpod-build-af-invoke-finalize-invoice-rpc-from-webhook.23](https://werft.gitpod-dev.com/job/gitpod-build-af-invoke-finalize-invoice-rpc-from-webhook.23) (with `.werft/` from `main`)
/hold as this is blocking the merge pool. It looks like it needs rebased onto `main`, to pick up the `gofmt` changes: https://github.com/gitpod-io/gitpod/pull/11958
> @andrew-farries Can we make it so that "unknown field" triggers at least a warning? That's the same mode we have in installer, and I think it's a good trade-off...
Going to move this back to draft given that I'm away from tomorrow and I haven't found an easy way to trigger warnings on unknown fields during unmarshalling yet.
closing as I don't have time to investigate this further.