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

factory: cleanup creation from package #5501

Merged
merged 1 commit into from
May 7, 2022

Conversation

abn
Copy link
Member

@abn abn commented Apr 26, 2022

This change ensures that generated toml file from packages contain all relevant metadata and handles groups correct.

Split from #5450

@abn abn requested a review from a team April 26, 2022 12:14
@abn abn marked this pull request as ready for review April 26, 2022 12:14
Copy link
Member

@radoering radoering left a comment

Choose a reason for hiding this comment

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

Maybe, there should be a unit test for create_pyproject_from_package?

@abn abn force-pushed the fix-factory-cleanup branch from 1754502 to bc9c90a Compare May 6, 2022 10:43
@abn abn requested a review from radoering May 6, 2022 10:44
@abn abn force-pushed the fix-factory-cleanup branch from bc9c90a to c4888d1 Compare May 6, 2022 11:33
@abn abn dismissed radoering’s stale review May 6, 2022 12:56

Review comments addressed.

@abn abn force-pushed the fix-factory-cleanup branch from c4888d1 to aa1e700 Compare May 7, 2022 11:17
Copy link
Member

@radoering radoering left a comment

Choose a reason for hiding this comment

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

Is extras still missing?

@abn abn force-pushed the fix-factory-cleanup branch from aa1e700 to fbb1721 Compare May 7, 2022 15:25
This change ensures that generated toml file from packages contain all
relevant metadata and handles groups correct.
@abn abn force-pushed the fix-factory-cleanup branch from fbb1721 to dfac6ee Compare May 7, 2022 15:31
@abn
Copy link
Member Author

abn commented May 7, 2022

Is extras still missing?

Even added a test now :D

@abn abn requested a review from radoering May 7, 2022 15:32
@abn abn dismissed radoering’s stale review May 7, 2022 15:33

Review feedback resolved.

@radoering radoering merged commit 402ca3b into python-poetry:master May 7, 2022
@abn abn deleted the fix-factory-cleanup branch May 7, 2022 15:45
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 29, 2024
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants