Rename error variable in go tests to err #8828 #11716

h4ck3r-04 · 2024-10-15T21:24:36Z

Renamed the variables from e to err as mentioned in the https://go.dev/doc/effective_go

According to the issue number #8828
the variables in go code were named using good and effective go practices.
Following the example link for reference mentioned in the issue the variables have been renamed.

renamed the variables from e to err as mentioned in the https://go.dev/doc/effective_go

packaging/docker/samples/golang/app/main.go

vishesh

Minor nit. LGTM

foundationdb-ci · 2024-10-20T06:12:33Z

Result of foundationdb-pr-clang-ide on Linux CentOS 7

Commit ID: 6a6a4e7
Duration 0:21:18
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-10-20T06:33:50Z

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

Commit ID: 6a6a4e7
Duration 0:42:31
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-10-20T06:41:17Z

Result of foundationdb-pr-macos on macOS Ventura 13.x

Commit ID: 6a6a4e7
Duration 0:49:57
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-10-20T06:44:18Z

Result of foundationdb-pr-clang-arm on Linux CentOS 7

Commit ID: 6a6a4e7
Duration 0:53:00
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-10-20T06:45:12Z

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

Commit ID: 6a6a4e7
Duration 0:53:54
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)
Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci · 2024-10-20T06:55:40Z

Result of foundationdb-pr-clang on Linux CentOS 7

Commit ID: 6a6a4e7
Duration 1:04:23
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-10-20T06:55:45Z

Result of foundationdb-pr on Linux CentOS 7

Commit ID: 6a6a4e7
Duration 1:04:26
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

recipes/go-recipes/priority.go

bindings/go/src/fdb/directory/directoryLayer.go

gm42 · 2024-10-20T10:11:53Z

bindings/go/src/fdb/doc.go

-        fmt.Printf("hello is now world, foo was: %s\n", string(ret.([]byte)))
-    }
+	    fmt.Printf("hello is now world, foo was: %s\n", string(ret.([]byte)))
+	}


Are these whitespace changes intentional?

This might be because of the default formatter. They were not intentional

i have removed these whitespaces.

bindings/go/src/fdb/fdb.go

bindings/go/src/fdb/tuple/tuple.go

contrib/mockkms/get_encryption_keys.go

recipes/go-recipes/table.go

…ctory_layer.go, to work on go formatting

bindings/go/src/fdb/directory/directory_layer.go

gm42 · 2024-10-21T07:16:07Z

There are still some e for element renamed to err, and the whitespaces change. Aside from that, LGTM; thanks!

renamed the variables from e to err as mentioned in the https://go.dev/doc/effective_go

foundationdb-ci · 2024-11-11T10:14:44Z

Result of foundationdb-pr-clang-arm on Linux CentOS 7

Commit ID: 5659f31
Duration 0:51:25
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-11-11T10:19:27Z

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

Commit ID: 5659f31
Duration 0:56:13
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)
Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci · 2024-11-11T10:20:40Z

Result of foundationdb-pr-clang on Linux CentOS 7

Commit ID: 5659f31
Duration 0:57:20
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-11-11T10:25:38Z

Result of foundationdb-pr on Linux CentOS 7

Commit ID: 5659f31
Duration 1:02:18
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

johscheuer

LGTM 👍 Thanks for the PR!

jzhou77 · 2024-12-03T17:34:40Z

There are conflicts blocking the merge. Can you fix them?

foundationdb-ci · 2024-12-07T04:06:28Z

Result of foundationdb-pr-clang-ide on Linux CentOS 7

Commit ID: 68acedd
Duration 0:21:35
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-07T04:22:36Z

Result of foundationdb-pr-macos-m1 on macOS Ventura 13.x

Commit ID: 68acedd
Duration 0:37:43
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-07T04:40:23Z

Result of foundationdb-pr-clang-arm on Linux CentOS 7

Commit ID: 68acedd
Duration 0:55:26
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-07T04:41:32Z

Result of foundationdb-pr-cluster-tests on Linux CentOS 7

Commit ID: 68acedd
Duration 0:56:36
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)
Cluster Test Logs zip file of the test logs (available for 30 days)

foundationdb-ci · 2024-12-07T04:43:30Z

Result of foundationdb-pr-macos on macOS Ventura 13.x

Commit ID: 68acedd
Duration 0:58:35
Result: ✅ SUCCEEDED
Error: N/A
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-07T04:45:12Z

Result of foundationdb-pr-clang on Linux CentOS 7

Commit ID: 68acedd
Duration 1:00:18
Result: ❌ FAILED
Error: Error while executing command: if python3 -m joshua.joshua list --stopped | grep ${ENSEMBLE_ID} | grep -q 'pass=10[0-9][0-9][0-9]'; then echo PASS; else echo FAIL && exit 1; fi. Reason: exit status 1
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-07T04:47:54Z

Result of foundationdb-pr on Linux CentOS 7

Commit ID: 68acedd
Duration 1:02:59
Result: ❌ FAILED
Error: Error while executing command: if python3 -m joshua.joshua list --stopped | grep ${ENSEMBLE_ID} | grep -q 'pass=10[0-9][0-9][0-9]'; then echo PASS; else echo FAIL && exit 1; fi. Reason: exit status 1
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-09T19:07:37Z

Result of foundationdb-pr-clang on Linux CentOS 7

Commit ID: 68acedd
Duration 1:01:28
Result: ❌ FAILED
Error: Error while executing command: if python3 -m joshua.joshua list --stopped | grep ${ENSEMBLE_ID} | grep -q 'pass=10[0-9][0-9][0-9]'; then echo PASS; else echo FAIL && exit 1; fi. Reason: exit status 1
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

foundationdb-ci · 2024-12-09T19:09:59Z

Result of foundationdb-pr on Linux CentOS 7

Commit ID: 68acedd
Duration 1:03:49
Result: ❌ FAILED
Error: Error while executing command: if python3 -m joshua.joshua list --stopped | grep ${ENSEMBLE_ID} | grep -q 'pass=10[0-9][0-9][0-9]'; then echo PASS; else echo FAIL && exit 1; fi. Reason: exit status 1
Build Log terminal output (available for 30 days)
Build Workspace zip file of the working directory (available for 30 days)

Rename error variable in go tests to err apple#8829

0e451b9

renamed the variables from e to err as mentioned in the https://go.dev/doc/effective_go

h4ck3r-04 changed the title ~~Rename error variable in go tests to err #8829~~ Rename error variable in go tests to err #8828 Oct 15, 2024

jzhou77 closed this Oct 16, 2024

jzhou77 reopened this Oct 16, 2024

jzhou77 requested a review from vishesh October 16, 2024 03:50

vishesh reviewed Oct 18, 2024

View reviewed changes

packaging/docker/samples/golang/app/main.go Outdated Show resolved Hide resolved

vishesh previously approved these changes Oct 18, 2024

View reviewed changes

Update packaging/docker/samples/golang/app/main.go

6a6a4e7

vishesh dismissed their stale review via 6a6a4e7 October 20, 2024 05:51

gm42 requested changes Oct 20, 2024

View reviewed changes

recipes/go-recipes/priority.go Outdated Show resolved Hide resolved