Skip to content

Conversation

@gouzil
Copy link
Member

@gouzil gouzil commented Jun 3, 2025

PR Category

User Experience

PR Types

Improvements

Description

see: astral-sh/ruff-pre-commit#123

@gouzil gouzil requested a review from SigureMo as a code owner June 3, 2025 08:04
@paddle-bot
Copy link

paddle-bot bot commented Jun 3, 2025

你的PR提交成功,感谢你对开源项目的贡献!
请关注后续CI自动化测试结果,详情请参考Paddle-CI手册
Your PR has been submitted. Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot paddle-bot bot added the contributor External developers label Jun 3, 2025
@gouzil gouzil requested review from SigureMo and Copilot and removed request for SigureMo June 3, 2025 08:04
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the pre-commit hook ID for Ruff from ruff to ruff-check to align with upstream changes.

  • Updated the hook ID in the pre-commit configuration.
Comments suppressed due to low confidence (1)

.pre-commit-config.yaml:65

  • After renaming the hook ID, ensure all occurrences of ruff in documentation, CI scripts, and other configuration files are updated to ruff-check to prevent broken references.
 - id: ruff-check 
Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMeow 🐾

Copy link
Member

@SigureMo SigureMo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTMeow 🐾

@SigureMo SigureMo merged commit 479b646 into PaddlePaddle:develop Jun 4, 2025
49 checks passed
@SigureMo SigureMo deleted the ruff/rename_ruff_check branch June 4, 2025 06:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

contributor External developers

2 participants