Why is this PR required? What issue does it fix?:
Improves code quality. Removes S1005 (Drop unnecessary use of the blank identifier) and opt.semgrep.err-nil-check
What this PR does?:
It removes superfluous nil err check before return and unnecessary assignment to the blank identifier.
Does this PR require any upgrade changes?:
No
If the changes in this PR are manually verified, list down the scenarios covered and commands you used for testing with logs:
NA
Any additional information for your reviewer?:
Mention if this PR is part of any design or a continuation of previous PRs
NA
Signed-off-by: Rahul Grover rahulgrover99@gmail.com
Why is this PR required? What issue does it fix?: Improves code quality. Removes
S1005(Drop unnecessary use of the blank identifier) andopt.semgrep.err-nil-checkWhat this PR does?: It removes superfluous nil err check before return and unnecessary assignment to the blank identifier.
Does this PR require any upgrade changes?: No
If the changes in this PR are manually verified, list down the scenarios covered and commands you used for testing with logs: NA
Any additional information for your reviewer?: Mention if this PR is part of any design or a continuation of previous PRs NA