aio-libs / aiosmtpd

A reimplementation of the Python stdlib smtpd.py based on asyncio.
https://aiosmtpd.aio-libs.org
Apache License 2.0
313 stars 95 forks source link

Migrate tool settings from setup.cfg to pyproject.toml #363

Open pepoluan opened 1 year ago

pepoluan commented 1 year ago

What do these changes do?

Practically all still-maintained projects are migrating to have a Single Source of Truth for their builds: pyproject.toml

This is in accordance with the guidelines in PEP518 and PEP621.

I have ascertained that all tools aiosmtpd uses -- both as dependency or for development -- now have support for putting their configuration in pyproject.toml (some needs a bit of workaround but works well). This left us with an empty setup.cfg file, which I removed because it's no longer useful (all setup information is now in pyproject.toml as well).

Are there changes in behavior for the user?

None if user has the latest pip and setuptools.

Related issue number

None.

Checklist

codecov[bot] commented 1 year ago

Codecov Report

Merging #363 (75cc9d5) into master (79c8671) will not change coverage. The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #363   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            7         7           
  Lines         1706      1706           
  Branches       310       310           
=========================================
  Hits          1706      1706           
Impacted Files Coverage Δ
aiosmtpd/__init__.py 100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

waynew commented 1 year ago

@pepoluan FYI I'm still super interested in this getting in. There is a merge conflict in __init__.py. I've (re)reviewed PRs newer and up to this one, and will probably review more tomorrow afternoon.