Skip to content

Add support for protocol v3 and add QuerySQL and ExecuteSQL client methods #1574

Add support for protocol v3 and add QuerySQL and ExecuteSQL client methods

Add support for protocol v3 and add QuerySQL and ExecuteSQL client methods #1574

Triggered via pull request December 18, 2024 20:12
Status Failure
Total duration 1m 23s
Artifacts

tests.yml

on: pull_request
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

51 errors and 1 warning
test (1.22, nightly, ubuntu-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, nightly, ubuntu-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, nightly, ubuntu-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.22, nightly, ubuntu-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, nightly, ubuntu-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, nightly, ubuntu-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.22, nightly, ubuntu-latest)
Process completed with exit code 1.
test (1.22, stable, ubuntu-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.22, stable, ubuntu-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, stable, ubuntu-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, stable, ubuntu-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.22, stable, ubuntu-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, stable, ubuntu-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, stable, ubuntu-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.22, stable, ubuntu-latest)
The operation was canceled.
test (1.23, stable, ubuntu-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.23, stable, ubuntu-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, stable, ubuntu-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, stable, ubuntu-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.23, stable, ubuntu-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, stable, ubuntu-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, stable, ubuntu-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.23, stable, ubuntu-latest)
The operation was canceled.
test (1.23, nightly, ubuntu-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.23, nightly, ubuntu-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, nightly, ubuntu-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, nightly, ubuntu-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.23, nightly, ubuntu-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, nightly, ubuntu-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, nightly, ubuntu-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.23, nightly, ubuntu-latest)
The operation was canceled.
test (1.23, stable, macos-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.23, stable, macos-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, stable, macos-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, stable, macos-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.23, stable, macos-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.23, stable, macos-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.23, stable, macos-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.23, stable, macos-latest)
The operation was canceled.
test (1.22, stable, macos-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.22, stable, macos-latest): internal/client/client.go#L444
exported: comment on exported method Client.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, stable, macos-latest): internal/client/client.go#L461
exported: comment on exported method Client.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, stable, macos-latest): internal/client/scriptflow.go#L78
rangeValCopy: each iteration copies 56 bytes (consider pointers or indexing) (gocritic)
test (1.22, stable, macos-latest): internal/client/transaction.go#L266
exported: comment on exported method Tx.ExecuteSQL should be of the form "ExecuteSQL ..." (revive)
test (1.22, stable, macos-latest): internal/client/transaction.go#L289
exported: comment on exported method Tx.QuerySQL should be of the form "QuerySQL ..." (revive)
test (1.22, stable, macos-latest): internal/client/warning.go#L36
exported: exported method Warning.Err should have comment or be unexported (revive)
test (1.22, stable, macos-latest)
The operation was canceled.
test (1.22, nightly, macos-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.22, nightly, macos-latest)
The operation was canceled.
test (1.23, nightly, macos-latest)
The job was canceled because "_1_22_nightly_ubuntu-late" failed.
test (1.23, nightly, macos-latest)
The operation was canceled.
test (1.22, nightly, ubuntu-latest)
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636