feat: Add batch signal support for contiguous item insertions and removals in EventedList - #422
feat: Add batch signal support for contiguous item insertions and removals in EventedList#422tlambert03 wants to merge 6 commits into
Conversation
…ovals in EventedList
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #422 +/- ##
=======================================
Coverage 99.75% 99.76%
=======================================
Files 23 23
Lines 2461 2520 +59
=======================================
+ Hits 2455 2514 +59
Misses 6 6 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Good stuff! This makes updates for big amounts of data so much easier. Thank you! |
|
Is there anything in particular you want to add before you merge, or is it ready to go? To me, it's been a great addition already and using it improves performance significantly. Would love for this to be merged and released 🙂 From the pending |
|
thanks for the ping @ndxmrb (and sorry again for the delay). I think it's fine. the only thing I'd like opinions on (from you, maybe @Czaki) is whether we like the names of the new signals |
|
Now that you say it, I like |
|
yeah i think i do too. will change and merge |
playing with explicit support of batch signals, re #419