Dominic Monroe

`sum` tag doesn't work with literal values

1

It appears it needs to use `compile-filter-body`

Behavior of children differs from React.Children

6

Attempting to reproduce https://codepen.io/SevereOverfl0w/pen/oOEvpY like-for-like does not currently work when doing: ``` (defn my-component [] (let [this (r/current-component)] (into [:div {:style {"backgroundColor" "red"}}] (map-indexed (fn [i child] (when (> i...

Redundant binding of _ at end of let

5

**Is your feature request related to a problem? Please describe.** It's quite common when working with react to use hooks. One hook is called "use-effect" and it's for creating side...

Detect import without require of records/interfaces

3

e.g. ``` (ns foo (:import [manifold.stream.core IEventSource])) ``` should result in an a lint failure telling me to require `manifold.stream.core` first. Related to #515

Support deprecated nss metadata warnings

3

**Is your feature request related to a problem? Please describe.** If I require a namespace that's marked deprecated (e.g. clojure.parallel would be after CLJ-706) then it would be nice to...

This breaks custom matchers, as mentioned in #187. #187 is not fixed by this because RoutePair generally needs fixing, but this makes it sufficiently optional for users.

e.g. :route-params is not mentioned in the README.

Coercion around a route causes custom match extension records to be lost (affects vhosts)

1

``` (defrecord A []) ((schema.coerce/coercer bidi.schema/RoutePair {}) [(->A) :match]) ``` Should result in an unchanged result, but actually results in `[{} :match]`, which _silently_ breaks the route structure. This seems...

A bit of batting practice

`=` with one argument makes no sense, I'm guessing there's a missing comparison here. https://github.com/juxt/bidi/blob/d1bfcc9c4fe247b8ada19cd7ee25acd81dd93f26/test/bidi/bidi_test.cljc#L297 https://github.com/juxt/bidi/blob/d1bfcc9c4fe247b8ada19cd7ee25acd81dd93f26/test/bidi/bidi_test.cljc#L303