feat: add periscope ECC bundle #28
ecc-tools[bot]Annotations
14 errors
|
integration
Process completed with exit code 2.
|
|
lint
issues found
|
|
lint:
internal/summarize/turns.go#L79
QF1002: could use tagged switch on m.Role (staticcheck)
|
|
lint:
internal/summarize/prompt.go#L314
QF1001: could apply De Morgan's law (staticcheck)
|
|
lint:
internal/llm/client.go#L217
S1016: should convert m (type Message) to msgBlock instead of using struct literal (staticcheck)
|
|
lint:
internal/summarize/prompt_test.go#L92
rangeint: for loop can be modernized using range over int (modernize)
|
|
lint:
internal/summarize/prompt.go#L412
minmax: user-defined min function is equivalent to built-in min and can be removed (modernize)
|
|
lint:
internal/signals/rewind.go#L179
minmax: if statement can be modernized using max (modernize)
|
|
lint:
internal/signals/compact.go#L312
stringsbuilder: using string += string in a loop is inefficient (modernize)
|
|
lint:
internal/signals/compact.go#L114
minmax: if statement can be modernized using max (modernize)
|
|
lint:
internal/server/context.go#L1406
stringsseq: Ranging over SplitSeq is more efficient (modernize)
|
|
Go Test (ubuntu-latest)
Process completed with exit code 1.
|
|
coverage
Process completed with exit code 1.
|
|
Go Test (windows-latest)
Process completed with exit code 1.
|