search

nteract / papermill

📚 Parameterize, execute, and analyze notebooks
http://papermill.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
5.99k stars 428 forks source link

Upgrade tests to moto v5 #779

Closed bnavigator closed 7 months ago

bnavigator commented 9 months ago

Moto v5 replaced mock_s3 with mock_aws

Supersedes #775

codecov[bot] commented 9 months ago

Codecov Report

Merging #779 (e6f60c5) into main (cb2eb37) will decrease coverage by 0.28%. Report is 4 commits behind head on main. The diff coverage is n/a.

:exclamation: Current head e6f60c5 differs from pull request most recent head 1185c15. Consider uploading reports for the commit 1185c15 to get more accurate results

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #779 +/- ## ========================================== - Coverage 91.54% 91.27% -0.28% ========================================== Files 17 17 Lines 1621 1627 +6 ========================================== + Hits 1484 1485 +1 - Misses 137 142 +5 ```
MSeal commented 7 months ago

Failure that's left is already fixed in main -- going to merge instead of trying to pull the base up to the PR