Skip to content

Forward compatibility for multiple readme files#4781

Closed
wagnerluis1982 wants to merge 1 commit into
python-poetry:masterfrom
wagnerluis1982:forward-compatibility-multiple-readme-files
Closed

Forward compatibility for multiple readme files#4781
wagnerluis1982 wants to merge 1 commit into
python-poetry:masterfrom
wagnerluis1982:forward-compatibility-multiple-readme-files

Conversation

@wagnerluis1982
Copy link
Copy Markdown
Contributor

@wagnerluis1982 wagnerluis1982 commented Nov 16, 2021

This makes poetry aware about the upcoming changes in python-poetry/poetry-core#248 in order to support declaration of multiple readme files in pyproject.toml

Relates: #873

This makes poetry aware about the upcoming changes in poetry-core in
order to support declaration of multiple readme files in pyproject.toml

Relates #873
Enable python-poetry/poetry-core#248
Comment thread tests/test_factory.py
assert package.license.id == "MIT"
assert (
package.readme.relative_to(fixtures_dir).as_posix()
single_readme.relative_to(fixtures_dir).as_posix()
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
single_readme.relative_to(fixtures_dir).as_posix()
# coment indicating why
getattr(package, "readmes", [package.readme]).relative_to(fixtures_dir).as_posix()

@wagnerluis1982
Copy link
Copy Markdown
Contributor Author

Closing as python-poetry/poetry-core#248 went into other direction, by ensuring retrocompatibiity.

@wagnerluis1982 wagnerluis1982 deleted the forward-compatibility-multiple-readme-files branch November 19, 2021 00:39
@github-actions
Copy link
Copy Markdown

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
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants