Switch clj-nil to not be considered a numeric type

Description

We should be able to get a warning like

We'd first need to fix CLJS-3085.

But with that, it seems as simple as having cljs.analyzer/numeric-type? return false instead of true for 'clj-nil by deleting the cond branch for it.

Environment

None

Activity

Show:
Roman Liutikov
April 24, 2020, 2:09 PM

Instead of fixing the macro ideally we should get in which in turn depends on

As a temporal fix we could silent warnings for unreachable code.

Roman Liutikov
April 24, 2020, 1:53 PM

Should still fix `cljs.spec.alpha/every`

Mike Fikes
August 20, 2019, 1:28 AM
Mike Fikes
August 20, 2019, 1:23 AM

Assignee

Roman Liutikov

Reporter

Mike Fikes

Labels

None

Approval

Incomplete

Patch

Code and Test

Priority

Minor