Skip to content

Conversation

@tklauser
Copy link
Contributor

There is no need to check err against nil, just return it.

There is no need to check err against nil, just return it.

Signed-off-by: Tobias Klauser <[email protected]>
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 72.322% when pulling c362597 on tklauser:skel-superfluous-err-nil-check into 6a92df8 on containernetworking:master.

Copy link
Member

@mars1024 mars1024 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm, thanks!

@tklauser
Copy link
Contributor Author

tklauser commented Nov 2, 2021

@mars1024 @mccv1r0 can this be merged given both of you approved?

@mars1024 mars1024 merged commit 1d9f32d into containernetworking:master Nov 8, 2021
@tklauser tklauser deleted the skel-superfluous-err-nil-check branch November 8, 2021 09:18
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.

4 participants