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

Update pyproject.toml #12520

Merged
merged 1 commit into from
May 28, 2024
Merged

Update pyproject.toml #12520

merged 1 commit into from
May 28, 2024

Conversation

GreatV
Copy link
Collaborator

@GreatV GreatV commented May 28, 2024

setuptools_scm may conflict with manifest.in, disable it for temporary.

@GreatV GreatV requested review from Liyulingyue and jzhang533 May 28, 2024 02:56
Copy link
Collaborator

@jzhang533 jzhang533 left a comment

Choose a reason for hiding this comment

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

LGTM

validated locally by running:
git clean -fxd && python -m build

there is a notice in setuptools_scm

setuptools-scm extracts Python package versions from git or hg metadata instead of declaring them as the version argument or in an SCM managed file.

Additionally, setuptools-scm provides setuptools with a list of files that are managed by the SCM
(i.e. it automatically adds all of the SCM-managed files to the sdist).
Unwanted files must be excluded via MANIFEST.in.

@jzhang533 jzhang533 merged commit 92c5984 into PaddlePaddle:main May 28, 2024
3 checks passed
@GreatV GreatV deleted the rm_setuptools_scm branch May 29, 2024 07:29
luzhongqiu pushed a commit to luzhongqiu/PaddleOCR that referenced this pull request Jun 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants