Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Cappl 374/load from empty #15659

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from
Draft

Cappl 374/load from empty #15659

wants to merge 2 commits into from

Conversation

MStreet3
Copy link
Contributor

Requires

Supports

Copy link
Contributor

AER Report: CI Core

aer_workflow , commit , Detect Changes , Clean Go Tidy & Generate , Scheduled Run Frequency , Flakeguard Root Project / Get Tests To Run , lint , Core Tests (go_core_tests) , Core Tests (go_core_tests_integration) , Core Tests (go_core_ccip_deployment_tests) , Flakeguard Deployment Project , Core Tests (go_core_race_tests) , Core Tests (go_core_fuzz) , Flakeguard Root Project / Run Tests (github.com/smartcontractkit/chainlink/v2/core/services/relay/evm/capabilities/workflow... , Flakey Test Detection , SonarQube Scan , Flakeguard Root Project / Report

1. Redundant return statement:[Golang Lint]

Source of Error:
core/services/workflows/syncer/workflow_registry.go:293:3: S1023: redundant `return` statement (gosimple)
		return
		^

Why: The error is caused by a redundant return statement in the code. The gosimple linter identifies this as an unnecessary statement that can be removed to simplify the code.

Suggested fix: Remove the redundant return statement from the specified line in workflow_registry.go. This will resolve the linter error and make the code cleaner.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant