Skip to content
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

setuptools 72.2.0 result in that MANIFEST.in is invalid #13670

Merged
merged 1 commit into from
Aug 15, 2024

Conversation

TingquanGao
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@cuicheng01 cuicheng01 left a comment

Choose a reason for hiding this comment

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

LGTM

@cuicheng01 cuicheng01 merged commit 543a4dd into PaddlePaddle:main Aug 15, 2024
3 of 4 checks passed
@GreatV
Copy link
Collaborator

GreatV commented Aug 15, 2024

为啥要固定版本呀?

@cuicheng01
Copy link
Collaborator

为啥要固定版本呀?

@GreatV CI拦截到的问题,setuptools的72.2.0版本(最新版)引入新问题,导致pip install .装的包丢失文件,官方可能在修复过程中了,修复之前先固定版本了

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 11, 2024
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