Proposal: rename `async def close()` to `async def aclose()` across aiohttp
オープン
enhancement
- 主要言語
- Python
- スター
- 16.5k
- フォーク
- 2.4k
- 平均マージ
- 17時間 22分
- マージ済み PR(30日)
- 212
説明
### Describe the solution you'd like
`contextlib.aclosing()` expects the object to provide an `aclose()` coroutine, which it awaits automatically. This makes `aclose` the de-facto naming convention for asynchronous close methods.
For consistency with this convention, I propose renaming `async def close()` to `async def aclose()` on all aiohttp classes that currently expose an async `close()`. Backward compatibility should be preserved: the old `close()` should keep working but emit a `DeprecationWarning` when called.
### Related component
Server and Client
### Code of Conduct
- [x] I agree to follow the aio-libs Code of Conduct
コントリビューションガイド
評価
この issue はまだ評価されていません。