Use io.CopyBuffer instead of io.ReadAll #184
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
|