Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

build: use lighter poetry-core #424

Merged
merged 1 commit into from
May 25, 2022
Merged

build: use lighter poetry-core #424

merged 1 commit into from
May 25, 2022

Conversation

branchvincent
Copy link
Contributor

@branchvincent branchvincent commented May 18, 2022

👋 Thanks for iredis, very happy user here!

Just noticed the project is still using the full poetry package as its build backend. There's now a preferred, lightweight poetry-core package. What's the benefit? To quote the project's readme:

Prior to the release of version 1.1.0, Poetry was a build as a project management tool that included a PEP 517 build backend. This was inefficient and time consuming in majority cases a PEP 517 build was required... This makes PEP 517 builds extremely fast for Poetry managed packages.

@laixintao
Copy link
Owner

Thanks! I didn't notice that. let me try.

@laixintao laixintao merged commit 1e42785 into laixintao:master May 25, 2022
@branchvincent branchvincent deleted the poetry-core branch May 25, 2022 12:24
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants