⬆Increase supported version of aiofiles to suppress warnings

[SnkSynthesis]

With Python 3.8 the usage of this @asyncio.coroutine in aiofiles 0.5.0 raises warnings since the @asyncio.coroutine has been deprecated since Python 3.8. This PR suppresses those warnings by increasing the aiofiles version to 0.7.0.

The lines (aiofiles/os.py) of aiofiles in version 0.5.0 (Notice the @asyncio.coroutine):

def wrap(func):
    @asyncio.coroutine
    @wraps(func)
    def run(*args, loop=None, executor=None, **kwargs):

The lines (aiofiles/os.py) of aiofiles in version 0.7.0 (Notice the async def):

def wrap(func):
    @wraps(func)
    async def run(*args, loop=None, executor=None, **kwargs):

[codecov]

Codecov Report

Merging #2899 (f00a27e) into master (69784e5) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #2899   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          410       410           
  Lines        10281     10281           
=========================================
  Hits         10281     10281           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 69784e5...f00a27e. Read the comment docs.

[github-actions]

📝 Docs preview for commit 6df49c9 at: https://6041a84dec518b4cc73f8876--fastapi.netlify.app

[wavenator]

Is there a reason why this PR is not merged yet? If any more work is needed, I'd be happy to jump in.

[wavenator]

In the meantime version 0.7.0 has been released. I recommend upgrading again.

[SnkSynthesis]

@wavenator The version of aiofiles has been increased to 0.7.0. Thank you for the recommendation.

[hongqn]

All reviewers have approved this pullreq. What's blocking the workflows?

[SnkSynthesis]

@hongqn, A maintainer (@tiangolo) needs to approve workflows and/or merge the pull request.

[zevisert]

I came looking to open this PR today, good to see some activity. I can pitch some commentary on the changelog from aiofiles.

0.7.0 (2021-05-17)
''''''''''''''''''
- Added the ``aiofiles.tempfile`` module for async temporary files.     <https://github.com/Tinche/aiofiles/pull/56>
- Switched to Poetry and GitHub actions.
- Dropped 3.5 support.

0.6.0 (2020-10-27)
''''''''''''''''''
- `aiofiles` is now tested on ppc64le.
- Added `name` and `mode` properties to async file objects.             <https://github.com/Tinche/aiofiles/pull/82>
- Fixed a DeprecationWarning internally.                                <https://github.com/Tinche/aiofiles/pull/75>
- Python 3.9 support and tests.

1. Although the changelog claims 0.6.0 is first to support python 3.9, 0.5.0 installs and functions just fine in 3.9. This is because aiofiles==0.5.0 used setuptools, but didn't specify python_requires.
2. The DepreciationWarning is what we're after here.
3. 0.7.0 drops python 3.5 support, but that's fine, we're >=3.6 here in FastAPI.
4. The note about switching to poetry in 0.7.0 reflects the drop of python 3.5 correctly. https://github.com/Tinche/aiofiles/blob/469c1a8b94ccccf25516706c88dde2e69ea45f3c/pyproject.toml#L9-L11
5. aiofiles has no dependencies in 0.5.0 or 0.6.0 or 0.7.0, so whatever version we choose here shouldn't conflict with any other packages.

[SnkSynthesis]

Thanks @zevisert!

[github-actions]

📝 Docs preview for commit f00a27e at: https://615c178297b64adca31ade5b--fastapi.netlify.app

[tiangolo]

Thanks for your contribution @SnkSynthesis! 🍰

And thanks for the discussion everyone, in particular @zevisert for all the investigation and report. 🕵️

I updated the range required to also include 0.5.0 as it still works, there's no need to force everyone to upgrade to the most recent one (and to avoid any possible conflicts with other packages depending on aiofiles).

On the other hand, just as a note, I'll soon upgrade to the latest Starlette, based on anyio, which provides the same functionality, so aiofiles will no longer be needed. 🎉

I'm making a release with these changes first just in case that's useful in the transition for anyone.

This is included in FastAPI 0.68.3, just released. 🎉

[SnkSynthesis]

Thanks @tiangolo 😄!