Skip to content

Conversation

@kurtmckee
Copy link
Contributor

These changes now allow the poetry check command to run successfully and without warnings.

As a part of this change, the docs extra was removed in favor of the Poetry docs dependency group.

  • The built sdist was checked and confirmed to still have the tests/ directory, as well as the py.typed file.
  • The built wheel was checked and confirmed to still have the py.typed file.

Note

GitHub's diff is rendering this differently than I would prefer. The coverage and mypy configurations were not edited in any way, despite GitHub's diff highlighting; rather, the Poetry configuration was moved to the top of the file and renamed to [project] (or [project.*]) and updated.

These changes now allow the `poetry check` command
to run successfully and without warnings.

As a part of this change, the `docs` extra was removed
in favor of the Poetry `docs` dependency group.

The built sdist was checked and confirmed to still have
the `tests/` directory, as well as the `py.typed` file.
The built wheel was checked and confirmed to still have
the `py.typed` file.
Copy link
Collaborator

@p1c2u p1c2u left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@p1c2u
Copy link
Collaborator

p1c2u commented Jun 28, 2025

@kurtmckee thanks for the contribution

@p1c2u p1c2u merged commit 9aac1fd into python-openapi:master Jun 28, 2025
11 checks passed
@kurtmckee kurtmckee deleted the use-pep-621-metadata branch June 29, 2025 10:49
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.

2 participants