search

clj-commons / manifold

A compatibility layer for event-driven abstractions
1.02k stars 106 forks source link

Wrong realized? version #212

Closed KingMob closed 2 years ago

KingMob commented 2 years ago

Manifold occasionally used clojure.core/realized? instead of its manifold.deferred/realized?. For the most part, this isn't a problem, since all the deferred classes implement IPending, but I'm experimenting with CompletableFuture support, and it would be a problem for that, since IPending is an interface which can't be extended to Java classes, and I can't change it to a protocol since it's part of core clojure.

KingMob commented 2 years ago

I also updated the docs a bit, set the Java target to 1.8, and slightly expanded the range of a flaky test.