Repository navigation
[refactor] Use match case instead of isinstance calls when the readability is better - #3431
Pierre-Sassoulas wants to merge 11 commits into
Conversation
Merging this PR will degrade performance by 4.78%1 warning
Warning Please fix the performance issues or acknowledge them on CodSpeed. Performance Changes
Tip Investigate this regression by commenting Comparing Footnotes
|
Codecov Report❌ Patch coverage is
❌ Your patch check has failed because the patch coverage (97.89%) is below the target coverage (100.00%). You can increase the patch coverage or adjust the target coverage. Additional details and impacted files@@ Coverage Diff @@
## main #3431 +/- ##
==========================================
+ Coverage 90.96% 91.18% +0.21%
==========================================
Files 93 93
Lines 11800 11804 +4
Branches 1941 1933 -8
==========================================
+ Hits 10734 10763 +29
+ Misses 747 734 -13
+ Partials 319 307 -12
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
|
Blocked by performance issue in match case fixed by Marc in python 3.13 if I remember correctly. |
Type of Changes
Description
Use match case when replacing isinstance calls result in a net positive line count.