Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Fix poetry pip-shims extras dependency#2078

Merged
timothycrosley merged 1 commit intoPyCQA:mainfrom
jooola:fix_poetry_validation
Jan 28, 2023
Merged

Fix poetry pip-shims extras dependency#2078
timothycrosley merged 1 commit intoPyCQA:mainfrom
jooola:fix_poetry_validation

Conversation

@jooola
Copy link
Contributor

@jooola jooola commented Jan 28, 2023

With poetry-core>=1.5.0, the dependencies in extras must be present in the main dependency group.

Fix #2077

With poetry-core>=1.5.0, the dependencies in extras must be present in the main dependency group.
@jooola jooola changed the title fix: poetry pip-shims extras format fix: poetry pip-shims extras dependency Jan 28, 2023
@jooola jooola changed the title fix: poetry pip-shims extras dependency Fix poetry pip-shims extras dependency Jan 28, 2023
Copy link

@darpham darpham left a comment

Choose a reason for hiding this comment

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

LGTM

@darpham
Copy link

darpham commented Jan 28, 2023

Alright, who's got the pager?

@mhsmith
Copy link

mhsmith commented Jan 28, 2023

To prevent something like this from happening again, maybe isort's pyproject.toml should pin a specific version of poetry-core.

@codecov
Copy link

codecov bot commented Jan 28, 2023

Codecov Report

Merging #2078 (0d219a6) into main (ae0cc1c) will not change coverage.
The diff coverage is n/a.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2078   +/-   ##
=======================================
  Coverage   99.22%   99.22%           
=======================================
  Files          38       38           
  Lines        3091     3091           
  Branches      747      747           
=======================================
  Hits         3067     3067           
  Misses         14       14           
  Partials       10       10           

@timothycrosley timothycrosley merged commit e833931 into PyCQA:main Jan 28, 2023
@jooola jooola deleted the fix_poetry_validation branch January 28, 2023 16:59
hugovk added a commit to hugovk/em-keyboard that referenced this pull request Jan 29, 2023
hugovk added a commit to hugovk/github-tools that referenced this pull request Jan 29, 2023
hugovk added a commit to hugovk/gutenberg-metadata that referenced this pull request Jan 29, 2023
hugovk added a commit to hugovk/hugovk that referenced this pull request Jan 29, 2023
hugovk added a commit to hugovk/lastfm-tools that referenced this pull request Jan 29, 2023
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.

Build from source (e.g. via pre-commit) is broken by new poetry version

4 participants