| 21:06 | <Michael Ficarra> | we really need to do the refactoring to get rid of optional AO parameters... |
| 21:07 | <shu> | is it blocking you on something? |
| 21:07 | <Michael Ficarra> | no I just hate it whenever I see it |
| 21:08 | <shu> | i think you just volunteered doing it then :) |
| 21:10 | <bakkot> | I started looking into it and it seemed like it would be ugly so I stopped |
| 21:10 | <bakkot> | optional parameters are kinda nice tbh |
| 21:10 | <Michael Ficarra> | oh I 100% do, I just need to not spend 100% of my editorial time doing PR reviews |
| 21:11 | <Michael Ficarra> | bakkot: were you duplicating the AO body or forwarding a default value to the AO with the longer parameter list? |
| 21:40 | <bakkot> | Michael Ficarra: I was just adding the default value at all callsites |
| 21:41 | <Michael Ficarra> | oh, yeah, depending on how many call sites there are, that makes sense too |
| 22:03 | <Michael Ficarra> | did we decide to allow "built-in generator function"s for the iterator helpers proposal? |
| 22:03 | <Michael Ficarra> | FYI, it looks like this: https://tc39.es/proposal-iterator-helpers/#sec-iteratorprototype.map |