Conversation
hugovk
left a comment
There was a problem hiding this comment.
Please could you add unit tests, and mention this in the release notes?
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
…te it is new Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
for more information, see https://pre-commit.ci
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
…[float] instead of Coords, radius to float Co-authored-by: Andrew Murray <3112309+radarhere@users.noreply.github.com>
I've added it to the release notes but am not quite sure how the unit tests work, especially the parametrization :/ |
|
I've created void4#1 with my suggestion for a test - I've copied To provide some information though, parametrization is essentially a loop - it runs the test with different items from a sequence. The idea is that if a particular scenario fails, pytest will clearly let us know which item from the parametrization sequence is the culprit. If you're interested more info, you can read up at https://docs.pytest.org/en/7.4.x/how-to/parametrize.html |
|
Thank you - merged :) |
|
Thank you! |
Resolves #8083
Implements the ImageDraw.circle feature as described and desired in the issue