REST: Fix PATCH method (#151)

* update goverter to 1.4.1 As that fixed a nil assignment override[1][2] which prevented us from emulating copy constructors[3]. The update also allows us to use generics. [1]: jmattheis/goverter#146 (comment) [2]: jmattheis/goverter#97 [3]: jmattheis/goverter#147 Signed-off-by: Isabella Basso do Amaral <[email protected]> Signed-off-by: Isabella do Amaral <[email protected]> * simplify converter utils using generics Signed-off-by: Isabella do Amaral <[email protected]> * server: update existing objects on PATCH Signed-off-by: Isabella do Amaral <[email protected]> --------- Signed-off-by: Isabella Basso do Amaral <[email protected]> Signed-off-by: Isabella do Amaral <[email protected]>
kubeflow · Jun 26, 2024 · 90ae8f4 · 90ae8f4
1 parent 1c46ce0
commit 90ae8f4
Show file tree

Hide file tree

Showing 11 changed files with 1,044 additions and 460 deletions.
diff --git a/Makefile b/Makefile
@@ -130,7 +130,7 @@ bin/golangci-lint:
 
 GOVERTER ?= ${PROJECT_BIN}/goverter
 bin/goverter:
-	GOBIN=$(PROJECT_PATH)/bin ${GO} install github.com/jmattheis/goverter/cmd/goverter@v1.1.1
+	GOBIN=$(PROJECT_PATH)/bin ${GO} install github.com/jmattheis/goverter/cmd/goverter@v1.4.1
 
 OPENAPI_GENERATOR ?= ${PROJECT_BIN}/openapi-generator-cli
 NPM ?= "$(shell which npm)"

diff --git a/internal/converter/generated/mlmd_openapi_converter.gen.go b/internal/converter/generated/mlmd_openapi_converter.gen.go