Skip to content

Removed a dead if-statement - #1476

Open
atombrella wants to merge 5 commits into
digitalocean:mainfrom
atombrella:feature/nilness_cleanup
Open

Removed a dead if-statement#1476
atombrella wants to merge 5 commits into
digitalocean:mainfrom
atombrella:feature/nilness_cleanup

Conversation

@atombrella

Copy link
Copy Markdown
Contributor

resourceDigitalOceanPartnerAttachmentCreate has one match

This was a match from running this .The nilness is used.

go vet -vettool $(which nilness) ./...

…reate

This was a match from running this .The `nilness` from analysis is used.
go vet -vettool $(which nilness) ./...
@atombrella

atombrella commented Dec 4, 2025

Copy link
Copy Markdown
Contributor Author

@SSharma-10 @do-joe This one also deserves some attention 😄

@atombrella
atombrella force-pushed the feature/nilness_cleanup branch from 7e4bc9d to 6dc1f04 Compare December 11, 2025 08:19
@do-joe

do-joe commented Dec 11, 2025

Copy link
Copy Markdown
Contributor

I am just a contributor and not a maintainer. The PR LGTM, but its very minor. Not sure if it will get the time of maintainers who are responsible for reviewing PRs for many of DO's OpenSource tools.

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.

2 participants