Skip to content

Use io.CopyBuffer instead of io.ReadAll #184

Use io.CopyBuffer instead of io.ReadAll

Use io.CopyBuffer instead of io.ReadAll #184

Triggered via pull request June 14, 2024 05:27
@k1LoWk1LoW
opened #79
use-copy
Status Failure
Total duration 1m 10s
Artifacts

ci.yml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

15 errors and 3 warnings
[gostyle] copybuf.go#L15: copybuf.go#L15
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://go.dev/wiki/CodeReviewComments#dont-panic )
[gostyle] copybuf.go#L10: copybuf.go#L10
[gostyle.getters] Function and method names should not use a "Get" or "get" prefix, unless the underlying concept uses the word "get" (e.g. an HTTP GET). Prefer starting the name with the noun directly, for example use "Counts" over "GetCounts". (ref: https://google.github.io/styleguide/go/decisions#getters ): getCopyBuf
Test: copybuf.go#L15
[gostyle.dontpanic] Don't use panic for normal error handling. Use error and multiple return values. (ref: https://go.dev/wiki/CodeReviewComments#dont-panic )
Test: copybuf.go#L10
[gostyle.getters] Function and method names should not use a "Get" or "get" prefix, unless the underlying concept uses the word "get" (e.g. an HTTP GET). Prefer starting the name with the noun directly, for example use "Counts" over "GetCounts". (ref: https://google.github.io/styleguide/go/decisions#getters ): getCopyBuf
Test
Process completed with exit code 1.
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Cannot open: File exists
Test
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v4. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Test
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
Test
Restore cache failed: Dependencies file is not found in /home/runner/work/rc/rc. Supported file pattern: go.sum