build: Require aiohttp v3.9.0+

nteract / papermill

📚 Parameterize, execute, and analyze notebooks

http://papermill.readthedocs.io/en/latest/

BSD 3-Clause "New" or "Revised" License

5.97k stars 429 forks source link

build: Require aiohttp v3.9.0+ #789

Closed matthewfeickert closed 7 months ago

matthewfeickert commented 7 months ago

What does this PR do?

Provide a lower bound of v3.9.0 for aiohttp. (c.f. https://github.com/nteract/papermill/issues/785#issuecomment-2032612755).
v3.9.0 was validated in PR https://github.com/nteract/papermill/pull/760

codecov[bot] commented 7 months ago

Codecov Report

Merging #789 (be2cf75) into main (cb2eb37) will decrease coverage by 0.26%. Report is 7 commits behind head on main. The diff coverage is n/a.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #789 +/- ## ========================================== - Coverage 91.54% 91.29% -0.26% ========================================== Files 17 17 Lines 1621 1631 +10 ========================================== + Hits 1484 1489 +5 - Misses 137 142 +5 ```

matthewfeickert commented 7 months ago

For whatever reason Codedcov is failing coverage, while it is passing on main, but that is safe to ignore as this is a 1-line diff.

matthewfeickert commented 7 months ago

Thanks for the fast review and merge, @MSeal!