feat: mimic some source 1 game events #760
This run and associated checks have been archived and are scheduled for deletion.
Learn more about checks retention
Annotations
5 errors and 1 warning
ci (1.18.x):
pkg/demoinfocs/datatables.go#L158
[golangci-lint] reported by reviewdog 🐶
`if p.isSource2()` has complex nested blocks (complexity: 10) (nestif)
Raw Output:
pkg/demoinfocs/datatables.go:158:3: `if p.isSource2()` has complex nested blocks (complexity: 10) (nestif)
if p.isSource2() {
^
|
ci (1.18.x):
pkg/demoinfocs/datatables.go#L112
[golangci-lint] reported by reviewdog 🐶
`if val.BoolVal()` has complex nested blocks (complexity: 5) (nestif)
Raw Output:
pkg/demoinfocs/datatables.go:112:4: `if val.BoolVal()` has complex nested blocks (complexity: 5) (nestif)
if val.BoolVal() {
^
|
ci (1.18.x):
pkg/demoinfocs/datatables.go#L1155
[golangci-lint] reported by reviewdog 🐶
missing cases in switch of type events.RoundEndReason: events.RoundEndReasonStillInProgress, events.RoundEndReasonVIPEscaped, events.RoundEndReasonVIPKilled, events.RoundEndReasonVIPNotEscaped (exhaustive)
Raw Output:
pkg/demoinfocs/datatables.go:1155:5: missing cases in switch of type events.RoundEndReason: events.RoundEndReasonStillInProgress, events.RoundEndReasonVIPEscaped, events.RoundEndReasonVIPKilled, events.RoundEndReasonVIPNotEscaped (exhaustive)
switch reason {
^
|
ci (1.18.x):
pkg/demoinfocs/datatables.go#L1154
[golangci-lint] reported by reviewdog 🐶
ST1023: should omit type common.Team from declaration; it will be inferred from the right-hand side (stylecheck)
Raw Output:
pkg/demoinfocs/datatables.go:1154:16: ST1023: should omit type common.Team from declaration; it will be inferred from the right-hand side (stylecheck)
var winner common.Team = common.TeamUnassigned
^
|
ci (1.18.x)
Process completed with exit code 1.
|
ci (1.18.x)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2, 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/
|