Skip to content
This repository has been archived by the owner on Jan 24, 2024. It is now read-only.

align codestyle to paddle #414

Merged
merged 2 commits into from
Oct 10, 2023
Merged

align codestyle to paddle #414

merged 2 commits into from
Oct 10, 2023

Conversation

SigureMo
Copy link
Member

@SigureMo SigureMo commented Oct 10, 2023

将 codestyle 对齐 Paddle,以便直接 copy 代码即可 merge

  • 移除 __all__,避免在 Paddle 暴露成 public API
  • 添加一个 noqa: W601,这是因为 ruff 和 flake8 差异导致的,暂时对齐 Paddle 的 flake8 行为,在之后 Paddle 也使用 ruff,并引入 RUF100 后会自动移除

@paddle-bot
Copy link

paddle-bot bot commented Oct 10, 2023

Thanks for your contribution!

@SigureMo SigureMo requested a review from gouzil October 10, 2023 07:43
Copy link
Collaborator

@feifei-111 feifei-111 left a comment

Choose a reason for hiding this comment

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

LGTM

@SigureMo SigureMo merged commit e33d698 into develop Oct 10, 2023
9 checks passed
@SigureMo SigureMo deleted the codestyle-align-to-paddle branch October 10, 2023 07:50
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants