Skip to content

[Misc] Fix staticcheck findings under pkg/#558

Open
heymrbox wants to merge 1 commit intomainfrom
weiwei-dev
Open

[Misc] Fix staticcheck findings under pkg/#558
heymrbox wants to merge 1 commit intomainfrom
weiwei-dev

Conversation

@heymrbox
Copy link
Copy Markdown
Collaborator

@heymrbox heymrbox commented Apr 3, 2026

What this PR does

Fix static analysis issues in pkg/

Why we need it

staticcheck reports multiple issues under pkg/.

Fixes #556

How to test

make test

Checklist

  • Tests added/updated (if applicable)
  • Docs updated (if applicable)
  • make test passes locally

@gemini-code-assist
Copy link
Copy Markdown
Contributor

gemini-code-assist bot commented Apr 3, 2026

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@github-actions github-actions bot added auth Authentication and authorization changes runtime Runtime configuration changes models Model configuration changes storage Storage provider changes inferenceservice InferenceService controller changes model-agent Model agent changes autoscaling Autoscaling (HPA/KEDA) changes controller Controller changes accelerator Accelerator class changes ingress Ingress/networking changes tests Test changes dependencies Dependency updates labels Apr 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@slin1237 slin1237 Awaiting requested review from slin1237 slin1237 is a code owner

@pallasathena92 pallasathena92 Awaiting requested review from pallasathena92 pallasathena92 is a code owner

@beiguo218 beiguo218 Awaiting requested review from beiguo218 beiguo218 is a code owner

@truddy0 truddy0 Awaiting requested review from truddy0 truddy0 is a code owner

@CatherineSue CatherineSue Awaiting requested review from CatherineSue CatherineSue is a code owner

@YouNeedCryDear YouNeedCryDear Awaiting requested review from YouNeedCryDear YouNeedCryDear is a code owner

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

accelerator Accelerator class changes auth Authentication and authorization changes autoscaling Autoscaling (HPA/KEDA) changes controller Controller changes dependencies Dependency updates inferenceservice InferenceService controller changes ingress Ingress/networking changes model-agent Model agent changes models Model configuration changes runtime Runtime configuration changes storage Storage provider changes tests Test changes

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Fix staticcheck findings in pkg/controller

1 participant

@heymrbox