-
Notifications
You must be signed in to change notification settings - Fork 227
infra: add golangci lint check #401
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
infra: add golangci lint check #401
Conversation
Signed-off-by: yuluo-yx <[email protected]>
Signed-off-by: yuluo-yx <[email protected]>
✅ Deploy Preview for vllm-semantic-router ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
👥 vLLM Semantic Team NotificationThe following members have been identified for the changed files in this PR and have been automatically assigned: 📁
|
Signed-off-by: yuluo-yx <[email protected]>
Signed-off-by: yuluo-yx <[email protected]>
Signed-off-by: yuluo-yx <[email protected]>
Signed-off-by: yuluo-yx <[email protected]>
Signed-off-by: yuluo-yx <[email protected]>
hi @rootfs If you have time, please take a look. thx |
* infra: add golangci lint check Signed-off-by: yuluo-yx <[email protected]> * infra: add golangci lint check Signed-off-by: yuluo-yx <[email protected]> * fi Signed-off-by: yuluo-yx <[email protected]> * fix Signed-off-by: yuluo-yx <[email protected]> * infra: add golang ci lint for precommit hook Signed-off-by: yuluo-yx <[email protected]> * fix test and build ci error Signed-off-by: yuluo-yx <[email protected]> * fix: fix precommit parallel golangci-lint run error Signed-off-by: yuluo-yx <[email protected]> * fix Signed-off-by: yuluo-yx <[email protected]> --------- Signed-off-by: yuluo-yx <[email protected]> Co-authored-by: Huamin Chen <[email protected]>
So far, there has been no feedback. This PR will complete this function.