[chore] Bump Go to 1.20 (#475)

* [chore] Update minimum go version to 1.20 Signed-off-by: Erik Westra <e.s.westra.95@gmail.com> * Update bump-go-1.20.yaml --------- Signed-off-by: Erik Westra <e.s.westra.95@gmail.com> Co-authored-by: Robert Pająk <pellared@hotmail.com>
open-telemetry · Dec 28, 2023 · 18ffba2 · 18ffba2
1 parent 41b2c05
commit 18ffba2
Show file tree

Hide file tree

Showing 41 changed files with 76 additions and 60 deletions.
diff --git a/.chloggen/bump-go-1.20.yaml b/.chloggen/bump-go-1.20.yaml
@@ -0,0 +1,16 @@
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: 'breaking'
+
+# The name of the component, or a single word describing the area of concern, (e.g. crosslink)
+component: all
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: "bump minimal Go version to 1.20"
+
+# One or more tracking issues related to the change
+issues: [474]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:
diff --git a/checkfile/go.mod b/checkfile/go.mod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/checkfile
 
-go 1.19
+go 1.20
 
 require github.com/stretchr/testify v1.8.4
 

diff --git a/chloggen/go.mod b/chloggen/go.mod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/chloggen
 
-go 1.19
+go 1.20
 
 require (
 	github.com/spf13/cobra v1.8.0

diff --git a/crosslink/README.md b/crosslink/README.md
@@ -141,6 +141,6 @@ for out-dated intra-repository Go modules.
 
 ### --go
 
- Go version applied when new `go.work` file is created (default "1.19").
+ Go version applied when new `go.work` file is created (default "1.20").
 
     crosslink work --go=1.20
diff --git a/crosslink/cmd/root.go b/crosslink/cmd/root.go
@@ -143,7 +143,7 @@ func init() {
 	comCfg.rootCommand.Flags().BoolVarP(&comCfg.runConfig.Prune, "prune", "p", false, "enables pruning operations on all go.mod files inside root repository")
 	comCfg.pruneCommand.Flags().StringSliceVar(&comCfg.excludeFlags, "exclude", []string{}, "list of comma separated go modules that crosslink will ignore in operations."+
 		"multiple calls of --exclude can be made")
-	comCfg.workCommand.Flags().StringVar(&comCfg.runConfig.GoVersion, "go", "1.19", "Go version applied when new go.work file is created")
+	comCfg.workCommand.Flags().StringVar(&comCfg.runConfig.GoVersion, "go", "1.20", "Go version applied when new go.work file is created")
 }
 
 // transform array slice into map

diff --git a/crosslink/go.mod b/crosslink/go.mod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink
 
-go 1.19
+go 1.20
 
 require (
 	github.com/google/go-cmp v0.6.0

diff --git a/crosslink/internal/crosslink_test.go b/crosslink/internal/crosslink_test.go
@@ -43,7 +43,7 @@ func TestCrosslink(t *testing.T) {
 			config:   DefaultRunConfig(),
 			expected: map[string][]byte{
 				"go.mod": []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 					")\n" +
@@ -52,13 +52,13 @@ func TestCrosslink(t *testing.T) {
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testZ => ./testZ\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ./testB"),
 				filepath.Join("testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB"),
 				filepath.Join("testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-					"go 1.19\n\n"),
+					"go 1.20\n\n"),
 			},
 		},
 		{
@@ -67,22 +67,22 @@ func TestCrosslink(t *testing.T) {
 			config:   DefaultRunConfig(),
 			expected: map[string][]byte{
 				"go.mod": []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ./testA\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ./testB"),
 				filepath.Join("testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot => ../"),
 				// b has req on root but not necessary to write out with current comparison logic
 				filepath.Join("testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ../testA\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot => ../\n\n"),
 			},
@@ -96,20 +96,20 @@ func TestCrosslink(t *testing.T) {
 			},
 			expected: map[string][]byte{
 				"go.mod": []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ./testA\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ./testB"),
 				filepath.Join("testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB"),
 				filepath.Join("testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-					"go 1.19\n\n"),
+					"go 1.20\n\n"),
 			},
 		},
 	}
@@ -188,20 +188,20 @@ func TestOverwrite(t *testing.T) {
 			},
 			expected: map[string][]byte{
 				"go.mod": []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ./testA\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ./testB"),
 				filepath.Join("testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB"),
 				filepath.Join("testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-					"go 1.19\n\n"),
+					"go 1.20\n\n"),
 			},
 		},
 		{
@@ -213,20 +213,20 @@ func TestOverwrite(t *testing.T) {
 			},
 			expected: map[string][]byte{
 				"go.mod": []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ../testA\n\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ./testB"),
 				filepath.Join("testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-					"go 1.19\n\n" +
+					"go 1.20\n\n" +
 					"require (\n\t" +
 					"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 					")\n" +
 					"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB"),
 				filepath.Join("testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-					"go 1.19\n\n"),
+					"go 1.20\n\n"),
 			},
 		},
 	}
@@ -352,20 +352,20 @@ func TestExclude(t *testing.T) {
 				// a mock_test_data_expected folder could be built instead of building expected files by hand.
 				modFilesExpected := map[string][]byte{
 					filepath.Join(tmpRootDir, "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot\n\n" +
-						"go 1.19\n\n" +
+						"go 1.20\n\n" +
 						"require (\n\t" +
 						"go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0\n" +
 						")\n" +
 						"replace go.opentelemetry.io/build-tools/crosslink/testroot/testA => ../testA\n\n" +
 						"replace go.opentelemetry.io/build-tools/excludeme => ../excludeme\n\n"),
 					filepath.Join(tmpRootDir, "testA", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testA\n\n" +
-						"go 1.19\n\n" +
+						"go 1.20\n\n" +
 						"require (\n\t" +
 						"go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0\n" +
 						")\n" +
 						"replace go.opentelemetry.io/build-tools/crosslink/testroot/testB => ../testB"),
 					filepath.Join(tmpRootDir, "testB", "go.mod"): []byte("module go.opentelemetry.io/build-tools/crosslink/testroot/testB\n\n" +
-						"go 1.19\n\n"),
+						"go 1.20\n\n"),
 				}
 
 				for modFilePath, modFilesExpected := range modFilesExpected {

diff --git a/crosslink/internal/mock_test_data/testCyclic/gomod b/crosslink/internal/mock_test_data/testCyclic/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
diff --git a/crosslink/internal/mock_test_data/testCyclic/testA/gomod b/crosslink/internal/mock_test_data/testCyclic/testA/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
diff --git a/crosslink/internal/mock_test_data/testCyclic/testB/gomod b/crosslink/internal/mock_test_data/testCyclic/testB/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot v1.0.0
diff --git a/crosslink/internal/mock_test_data/testExclude/gomod b/crosslink/internal/mock_test_data/testExclude/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testExclude/testA/gomod b/crosslink/internal/mock_test_data/testExclude/testA/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testExclude/testB/gomod b/crosslink/internal/mock_test_data/testExclude/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testNoOverwrite/gomod b/crosslink/internal/mock_test_data/testNoOverwrite/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testNoOverwrite/testA/gomod b/crosslink/internal/mock_test_data/testNoOverwrite/testA/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
diff --git a/crosslink/internal/mock_test_data/testNoOverwrite/testB/gomod b/crosslink/internal/mock_test_data/testNoOverwrite/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testOverwrite/gomod b/crosslink/internal/mock_test_data/testOverwrite/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testOverwrite/testA/gomod b/crosslink/internal/mock_test_data/testOverwrite/testA/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
diff --git a/crosslink/internal/mock_test_data/testOverwrite/testB/gomod b/crosslink/internal/mock_test_data/testOverwrite/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testPrune/gomod b/crosslink/internal/mock_test_data/testPrune/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require (
     go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0

diff --git a/crosslink/internal/mock_test_data/testSimple/gomod b/crosslink/internal/mock_test_data/testSimple/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testSimple/testA/gomod b/crosslink/internal/mock_test_data/testSimple/testA/gomod
@@ -1,5 +1,5 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
diff --git a/crosslink/internal/mock_test_data/testSimple/testB/gomod b/crosslink/internal/mock_test_data/testSimple/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testSimplePrune/gomod b/crosslink/internal/mock_test_data/testSimplePrune/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testA v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testSimplePrune/testA/gomod b/crosslink/internal/mock_test_data/testSimplePrune/testA/gomod
@@ -1,6 +1,6 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
 
 require go.opentelemetry.io/build-tools/crosslink/testroot/testB v1.0.0
 

diff --git a/crosslink/internal/mock_test_data/testSimplePrune/testB/gomod b/crosslink/internal/mock_test_data/testSimplePrune/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testWork/go.work b/crosslink/internal/mock_test_data/testWork/go.work
@@ -1,4 +1,4 @@
-go 1.19
+go 1.20
 
 // existing valid use statements under root should remain
 use ./testA

diff --git a/crosslink/internal/mock_test_data/testWork/gomod b/crosslink/internal/mock_test_data/testWork/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testWork/testA/gomod b/crosslink/internal/mock_test_data/testWork/testA/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testA
 
-go 1.19
+go 1.20
diff --git a/crosslink/internal/mock_test_data/testWork/testB/gomod b/crosslink/internal/mock_test_data/testWork/testB/gomod
@@ -1,3 +1,3 @@
 module go.opentelemetry.io/build-tools/crosslink/testroot/testB
 
-go 1.19
+go 1.20