chore(ci): align pre-commit config with Kubeflow-SDK Introduced check-added-large-files#71
Conversation
…-added-large-files Signed-off-by: Raghul-M <raghul.m1430@gmail.com>
|
🎉 Welcome to the Kubeflow MCP Server! 🎉 Thanks for opening your first PR! We're happy to have you as part of our community 🚀 Here's what happens next:
Join the community:
Feel free to ask questions in the comments if you need any help or clarification! |
There was a problem hiding this comment.
Pull request overview
Aligns pre-commit checks with Kubeflow SDK and completes issue #60.
Changes:
- Adds large-file validation.
- Replaces the legacy
ruffhook ID withruff-check.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
/ok-to-test Looks good, closes the remaining #60 gaps . |
|
@Raghul-M can you check unit test for pre-commit failing 👀 |
Signed-off-by: Raghul-M <raghul.m1430@gmail.com>
fixed |
|
/lgtm |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: abhijeet-dhumal The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
Update
.pre-commit-config.yamlto match kubeflow/sdk and close the remaining gaps from #60:check-added-large-filesruffhook id toruff-checkNote: the config file already existed; this PR completes the issue requirements rather than adding it from scratch.
Related Issue
Fixes #60
Checklist
make test-python)make verify)git commit -s)Testing
Ran
uv run pre-commit run --all-files(ormake verify) after the hook id change.