-
Notifications
You must be signed in to change notification settings - Fork 9
Open
Labels
Agenda+To be discussed in next breakout. Remove when actually discussed and resolved.To be discussed in next breakout. Remove when actually discussed and resolved.needs consensus
Milestone
Description
Split out of #75 (comment)
Reasons to special-case:
- You pretty much never want symbols there, so
requires constructor implements protocol { ... }is a footgun. - Without the special-casing
requires constructorcreates aprotocol.constructorsymbol, but inProtocolinstances,protocol.constructorshould point toProtocol. And whileprotocol.prototypewould be blank, it does break assumptions about whatobj.prototypeis, and prevents us from usingprotocol.prototypefor inheritance. - Defining class statics is a common enough use case that it originally had dedicated syntax
Reasons to not special-case:
- Slight increase in mental model / handling complexity
I’m personally moderately strongly in favor of special-casing, but @michaelficarra and @ljharb had reservations. Could you elaborate what these were? (feel free to edit this comment so we have all the pros and cons in one place)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Agenda+To be discussed in next breakout. Remove when actually discussed and resolved.To be discussed in next breakout. Remove when actually discussed and resolved.needs consensus