Skip to content
For each commit and PR

Add Unmarshal(cfg string) to VendorConfigManager #149

Sign in to view logs

Add Unmarshal(cfg string) to VendorConfigManager

Add Unmarshal(cfg string) to VendorConfigManager #149

Triggered via pull request July 18, 2024 11:40
@splaspoodsplaspood
synchronize #17
vcm-unmarshal-0
Status Failure
Total duration 28s
Artifacts –

ci.yaml

on: pull_request
lint
18s
lint
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 1 warning
lint: config/asrockrack.go#L51
cannot use asrr (variable of type *asrockrackVendorConfig) as type VendorConfigManager in return statement:
lint: config/asrockrack.go#L120
not enough return values
lint: config/dell.go#L57
cannot use dell (variable of type *dellVendorConfig) as type VendorConfigManager in return statement:
lint: config/dell.go#L144
not enough return values
lint: config/asrockrack.go#L1
: # github.com/bmc-toolbox/common/config
lint: config/asrockrack.go#L51
cannot use asrr (variable of type *asrockrackVendorConfig) as type VendorConfigManager in return statement:
lint: config/asrockrack.go#L120
not enough return values
lint: config/dell.go#L57
cannot use dell (variable of type *dellVendorConfig) as type VendorConfigManager in return statement:
lint: config/dell.go#L144
not enough return values
lint
issues found
lint
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3, actions/setup-go@v3, golangci/golangci-lint-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/