Fork me on GitHub
#clojurescript
<
2019-06-13
>
tomjkidd02:06:29

Hey, trying to migrate from clojurescript 1.10.339 to 1.10.520 and am running into an issue: We had a spec defined like the following

(s/def ::x (s/* vector?))
We would use a set of vectors as the argument and it would be valid, but now…
(s/explain-data ::x #{[:a]})
#:cljs.spec.alpha{:problems
                  [{:path [],
                    :pred
                    (cljs.core/fn
                     [%]
                     (cljs.core/or
                      (cljs.core/nil? %)
                      (cljs.core/sequential? %))),
                    :val #{[:a]},
                    :via [:cljs.user/x],
                    :in []}], 
                  :spec :cljs.user/x, 
                  :value #{[:a]}}
Is this a bug, or an intentional change?

Alex Miller (Clojure team)02:06:39

answered in #clojure-spec

Chris Swanson13:06:22

anyone used re-graph with re-frame? I've got it working fine with queries, but ws subs don't work, they throw errors like this repeatedly:

Ignoring graphql-ws event  Object {
  "_hash": -1918427164,
  "cljs$lang$protocol_mask$partition0$": 2153775105,
  "cljs$lang$protocol_mask$partition1$": 4096,
  "fqn": "re-graph.internals/default",
  "name": "default",
  "ns": "re-graph.internals",
}  -  ka
For context , this is an expo app using hasura /pg as graphql backend

Matt Butler14:06:30

Anyone measure test coverage in cljs? Cloverage doesn't seem to have support.

Matt Butler07:06:46

Thanks, I'll take a look.

Oliver George09:06:54

I’d be interested to hear how you go. I’d love to have a useful coverage tool for cljs.

agigao17:06:07

Hi Clojurians, I’m trying to update/remove class from js object. I do get a list of nodes through .getElementsByClassName

chancerussell22:06:48

@U2HBNQQBE Not sure the problem you’re having, but if it is related to filtering or iterating across the result of .getElementsByClassName, you may need to call cljs.core/array-seq

chancerussell22:06:14

The NodeList object returned by browser dom apis isn’t really an array