Skip to content

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Sep 15, 2022

PR types

Others

PR changes

Others

Describe

移除 .yaml(含 .yml)文件和 .hook 文件的 trailing-whitespace

以及 3 个 python 增量问题

SigureMo added a commit to cattidea/Paddle that referenced this pull request Sep 15, 2022
@SigureMo SigureMo force-pushed the trailing-whitespace-yaml-hook branch from 846bc79 to 90b8448 Compare September 15, 2022 17:28
@paddle-bot-old paddle-bot-old bot added the contributor External developers label Sep 15, 2022
@luotao1 luotao1 self-assigned this Sep 16, 2022
Copy link
Contributor

@luotao1 luotao1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@Ligoml Ligoml left a comment

Choose a reason for hiding this comment

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

LGTM

@luotao1 luotao1 merged commit 95c659f into PaddlePaddle:develop Sep 19, 2022
luotao1 pushed a commit that referenced this pull request Sep 19, 2022
* adjust pre-commit config

* incremental trailing whitespace

* revert python increment, fix in #46080

* add .yaml and .hook suffix to pre-commit config, test=document_fix

* add yml suffix
@SigureMo SigureMo deleted the trailing-whitespace-yaml-hook branch September 19, 2022 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
contributor External developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants