Skip to content

Merge pull request #80 from attestantio/rework-api #447

Merge pull request #80 from attestantio/rework-api

Merge pull request #80 from attestantio/rework-api #447

Triggered via push October 21, 2023 08:34
Status Failure
Total duration 1m 15s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention
Fit to window
Zoom out
Zoom in

Annotations

10 errors
lint: http/parameters.go#L35
interface method apply must have all named params (inamedparam)
lint: auto/parameters.go#L31
interface method apply must have all named params (inamedparam)
lint: mock/parameters.go#L32
interface method apply must have all named params (inamedparam)
lint: http/events.go#L186
return with no blank line before (nlreturn)
lint: spec/phase0/beaconstate.go#L132
fmt.Sprintf can be replaced with faster strconv.FormatUint (perfsprint)
lint: spec/phase0/beaconstate.go#L142
fmt.Sprintf can be replaced with faster strconv.FormatUint (perfsprint)
lint: spec/phase0/eth1data.go#L54
fmt.Sprintf can be replaced with faster strconv.FormatUint (perfsprint)
lint: spec/bellatrix/executionaddress.go#L30
fmt.Sprintf can be replaced with faster hex.EncodeToString (perfsprint)
lint: api/v1/genesis.go#L45
fmt.Sprintf can be replaced with faster strconv.FormatInt (perfsprint)
lint: api/v1/validatorregistration.go#L60
fmt.Sprintf can be replaced with faster strconv.FormatInt (perfsprint)