Skip to content

sort package name in extras to make it reproducible#280

Merged
finswimmer merged 1 commit into
python-poetry:masterfrom
ali5h:poetry-issue-5125
Feb 4, 2022
Merged

sort package name in extras to make it reproducible#280
finswimmer merged 1 commit into
python-poetry:masterfrom
ali5h:poetry-issue-5125

Conversation

@ali5h
Copy link
Copy Markdown
Contributor

@ali5h ali5h commented Feb 3, 2022

@ali5h ali5h force-pushed the poetry-issue-5125 branch from 347bbeb to e6145c4 Compare February 3, 2022 03:24
Copy link
Copy Markdown
Member

@finswimmer finswimmer left a comment

Choose a reason for hiding this comment

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

Thanks a lot for your contribution 👍

Can you please add some tests that cover the changes?

@ali5h ali5h force-pushed the poetry-issue-5125 branch from e6145c4 to ccc8a1f Compare February 3, 2022 19:43
@ali5h
Copy link
Copy Markdown
Contributor Author

ali5h commented Feb 3, 2022

added

@ali5h ali5h force-pushed the poetry-issue-5125 branch from ccc8a1f to 6bb8add Compare February 3, 2022 21:13
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Feb 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 2 Code Smells

No Coverage information No Coverage information
3.8% 3.8% Duplication

@ali5h ali5h requested a review from finswimmer February 3, 2022 22:11
Copy link
Copy Markdown
Member

@finswimmer finswimmer left a comment

Choose a reason for hiding this comment

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

Thanks a lot. Looks good to me 🚀

@finswimmer finswimmer merged commit 0113a5a into python-poetry:master Feb 4, 2022
@ali5h ali5h deleted the poetry-issue-5125 branch February 4, 2022 06:14
@finswimmer finswimmer mentioned this pull request Feb 9, 2022
DavidVujic pushed a commit to DavidVujic/poetry-core that referenced this pull request Mar 26, 2022
bostonrwalker pushed a commit to bostonrwalker/poetry-core that referenced this pull request Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

package.extras is not reproducible

2 participants