No worries. Webpack made me sad for like 3 weeks so this was relatively minor in comparison ;) I'm not familiar with Opaque. Link?
-
-
Replying to @chriseppstein @tomdale
Opaque is defined in @glimmer/util as https://github.com/glimmerjs/glimmer-vm/blob/7f40b03952e9bd11f517279cdfe61b4888e3ef2d/packages/%40glimmer/interfaces/lib/core.d.ts#L1 … It is generally preferred in the Glimmer projects over `any`.
1 reply 0 retweets 0 likes -
With that said, I’m not sure it would have prevented your bug. I’m guessing the resolution just didn’t match your expectations?
1 reply 0 retweets 0 likes -
Huh. It might have because it would have forced an explicit cast. But why not use a generic?
1 reply 0 retweets 0 likes -
Replying to @chriseppstein @tomdale
Yeah, it’s no generic (not yet anyway). I’m not clear on the history or future of Opaque like perhaps
@wycats or@krisselden.1 reply 0 retweets 0 likes -
Replying to @dgeb @chriseppstein and
We should use Opaque for public returns.
2 replies 0 retweets 0 likes -
I still don't understand why this isn't a generically typed Interface.
2 replies 0 retweets 0 likes -
Replying to @chriseppstein @wycats and
I agree it should be generic
https://github.com/glimmerjs/glimmer-vm/blob/more-aot/packages/%40glimmer/interfaces/lib/di.d.ts …2 replies 0 retweets 3 likes -
Replying to @chancancode @chriseppstein and
A generic that can't infer anything is no better than a cast, it's the same, if it took a mapping between keys and type then it could
1 reply 0 retweets 1 like -
Replying to @krisselden @chancancode and
It seems like this should be possible in 2.4?
1 reply 0 retweets 0 likes
We worked on it. It's non trivial. But if you look at the code in the test environment you'll see we made some progress.
-
-
Replying to @wycats @krisselden and
I know all y'all care about this stuff. Thanks for filling me in. I'll wrap the API for now.
0 replies 0 retweets 2 likesThanks. Twitter will use this to make your timeline better. UndoUndo
-
Loading seems to be taking a while.
Twitter may be over capacity or experiencing a momentary hiccup. Try again or visit Twitter Status for more information.