Move mocked_compute to conftest.py
#6564
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #6564 +/- ##
==========================================
+ Coverage 89.91% 89.93% +0.02%
==========================================
Files 90 90
Lines 24153 24166 +13
Branches 4496 4502 +6
==========================================
+ Hits 21716 21733 +17
+ Misses 1678 1672 -6
- Partials 759 761 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Co-authored-by: DarkVoyager11 <[email protected]>
Co-authored-by: DarkVoyager11 <[email protected]>
docs/src/whatsnew/latest.rst
Outdated
| @@ -147,7 +147,10 @@ This document explains the changes made to Iris for this release | |||
| #. `@bjlittle`_ added support for `Trusted Publishing`_ of source distributions | |||
| and binary wheels to PyPI and Test PyPI. (:pull:`6543`) | |||
|
|
|||
|
|
|||
| #. `@ESadek-MO`_ moved `@rcomer`_ s `mocked_compute` testing fixture into the `conftest.py` for | |||
There was a problem hiding this comment.
Bit of a nitpick but I believe this should be `@rcomer`_'s .
There was a problem hiding this comment.
There does need to be a space here I THINK, otherwise formatting goes beserk. I'll add the apostrophe in though.
There was a problem hiding this comment.
Looks like we've been able to get away with just an apostrophe before, might just be having a letter after an underscore which it doesn't like.
iris/docs/src/whatsnew/3.2.rst
Line 317 in 13e3d9f
There was a problem hiding this comment.
Huh. Interesting, I'd never noticed that. I'll push that up.
🚀 Pull Request
Description
Closes #5704.
Consult Iris pull request check list
Add any of the below labels to trigger actions on this PR: