01:25
<sideshowbarker>
comments on https://github.com/mdn/content/pull/15271 welcome either there or here
08:20
<sideshowbarker>
looking at https://github.com/mdn/content/issues/15203, should Symbol also be included in the list String, Array, TypedArray, Map, and Set are all built-in iterables, because each of their prototype objects implements an @@iterator method.“? (I guess the list is also missing Intl.Segments)
09:39
<Ashley Claymore>

Symbol doesn’t implement @@iterator.

Symbol.iterator in Symbol.prototype === false

09:48
<Ashley Claymore>

Though I did make this mistake recently when I wrongly wrote

isWellKnownSymbol = v => typeof v === 'symbol' && Reflect.ownKeys(Symbol).includes(v);

🤦🏻‍♂️

16:48
<ljharb>
it’d also have to be a frozen property, and you’d want to cache that list as early as possible
17:14
<Ashley Claymore>
yep, I was miles off. Got it in the end: https://gist.github.com/acutmore/0a5bf0f35785edfe3c17e1f6d05e8921