Issue 1007

[jackdewinter]

No description provided.

[sourcery-ai]

Hey @jackdewinter - I've reviewed your changes and they look great!

General suggestions:

• Consider simplifying the implementation to reduce complexity and improve maintainability.
• Ensure that new methods and logic introduced are well-documented to aid in future development.
• Review the necessity and implementation of newly introduced methods to ensure they align with the project's coding standards and practices.

Here's what I looked at during the review

• 🟡 General issues: 5 issues found
• 🟢 Security: all looks good
• 🟢 Testing: all looks good
• 🟡 Complexity: 3 issues found
• 🟢 Docstrings: all looks good

Thanks for using Sourcery. We offer it for free for open source projects and would be very grateful if you could help us grow. If you like it, would you consider sharing Sourcery on your favourite social media? ✨

Share Sourcery

• X
• Mastodon
• LinkedIn
• Facebook

---

Help me be more useful! Please click 👍 or 👎 on each comment to tell me if it was helpful.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (32ca849) to head (5c2672b).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##              main     #1020    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files          189       189            
  Lines        19456     19585   +129     
  Branches      2419      2444    +25     
==========================================
+ Hits         19456     19585   +129     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.