canonical / paas-app-charmer

2 stars 2 forks source link

Move setup.py to pyproject.toml #2

Closed lengau closed 7 months ago

lengau commented 7 months ago

Overview

Migrates the metadata stored in setup.py to pyproject.toml. A side-effect of this change is that installs from the sdist work, as requirements.txt is included. (You can check this yourself by running python -m build with and without this change.)

Rationale

While setup.py is not actually deprecated, modern tools tend to expect this data in pyproject.toml.

Juju Events Changes

None

Module Changes

None

Library Changes

None ### Checklist - [x] The [charm style guide](https://juju.is/docs/sdk/styleguide) was applied - [x] The [contributing guide](https://github.com/canonical/is-charms-contributing-guide) was applied - [x] The changes are compliant with [ISD054 - Managing Charm Complexity](https://discourse.charmhub.io/t/specification-isd014-managing-charm-complexity/11619) - [x] The documentation is generated using `src-docs` - [x] The documentation for charmhub is updated - [ ] The PR is tagged with appropriate label (`urgent`, `trivial`, `complex`)

I don't have access to tag this PR :-)