Skip to content

FEATURE: improve trade/order converter #5616

FEATURE: improve trade/order converter

FEATURE: improve trade/order converter #5616

Triggered via pull request August 8, 2024 09:58
Status Success
Total duration 11m 11s
Artifacts

go.yml

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

Annotations

13 warnings
build (6.2, 1.21)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
build (6.2, 1.21)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v2, codecov/codecov-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
build (6.2, 1.21)
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
build (6.2, 1.21)
Unexpected input(s) 'reporter', 'fail_on_error', valid inputs are ['entryPoint', 'args', 'config', 'exclude', 'path']
build (6.2, 1.21): pkg/cmd/strategy/builtin.go#L5
a blank import should be only in a main or test package, or have a comment justifying it
build (6.2, 1.21): pkg/style/colors.go#L1
should have a package comment
build (6.2, 1.21): cmd/bbgo/main.go#L1
should have a package comment
build (6.2, 1.21): pkg/testing/httptesting/client.go#L36
func HttpClientFromFile should be HTTPClientFromFile
build (6.2, 1.21): pkg/style/pnl.go#L7
exported var LossEmoji should have comment or be unexported
build (6.2, 1.21): pkg/style/table.go#L8
exported function NewDefaultTableStyle should have comment or be unexported
build (6.2, 1.21): pkg/slack/slackstyle/style.go#L1
should have a package comment
build (6.2, 1.21): pkg/testing/httptesting/response.go#L29
func BuildResponseJson should be BuildResponseJSON
build (6.2, 1.21): pkg/testing/httptesting/transport.go#L87
func MockWithJsonReply should be MockWithJSONReply