Exclude idl/ as a source of Go code (jaegertracing#6591)

## Which problem is this PR solving? - Part of jaegertracing#6494 - Since `jaeger-idl` repo now includes Go code, as soon as we update the submodule here it's treated as a real source of Go code, including by tools like `go fmt`, etc. ## Description of the changes - Exclude `idl/` from ALL_SOURCES, so that `make fmt` does not apply to it - Bump `idl` submodule to latest ## How was this change tested? - CI Signed-off-by: Yuri Shkuro <[email protected]>
Manik2708 · Jan 22, 2025 · b689a86 · b689a86
1 parent cbad04b
commit b689a86
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Makefile b/Makefile
@@ -29,6 +29,7 @@ ALL_SRC = $(shell find . -name '*.go' \
 				   -not -name 'mocks*' \
 				   -not -name '*.pb.go' \
 				   -not -path './vendor/*' \
+				   -not -path './idl/*' \
 				   -not -path './internal/tools/*' \
 				   -not -path './docker/debug/*' \
 				   -not -path '*/mocks/*' \

diff --git a/idl b/idl
+1 −1		.github/workflows/ci-lint-test.yml
+8 −0		.github/workflows/ci-unit-tests.yml
+1 −1		.gitignore
+81 −34		Makefile
+0 −71		Makefile.Protobuf.mk
+2 −2		README.md
+2 −1		go.mod
+2 −0		go.sum
+19 −0		model/v1/dependencies.go
+19 −0		model/v1/dependencies_test.go
+6 −0		model/v1/doc.go
+25 −0		model/v1/hash.go
+49 −0		model/v1/hash_test.go
+77 −0		model/v1/ids_proto_test.go
+246 −0		model/v1/keyvalue.go
+158 −0		model/v1/keyvalue_test.go
+158 −0		model/v1/keyvalues_test.go
+14 −0		model/v1/package_test.go
+35 −0		model/v1/process.go
+94 −0		model/v1/process_test.go
+231 −0		model/v1/prototest/model_test.pb.go
+30 −0		model/v1/prototest/model_test.proto
+101 −0		model/v1/sort.go
+129 −0		model/v1/sort_test.go
+141 −0		model/v1/span.go
+17 −0		model/v1/span_pkg_test.go
+333 −0		model/v1/span_test.go
+61 −0		model/v1/spanref.go
+54 −0		model/v1/spanref_test.go
+37 −0		model/v1/time.go
+31 −0		model/v1/time_test.go
+23 −0		model/v1/trace.go
+57 −0		model/v1/trace_test.go