jonase / eastwood

Clojure lint tool
1.09k stars 66 forks source link

Remove some `println`s #381

Closed vemv closed 3 years ago

vemv commented 3 years ago

Remove some printlns

On certain inputs, they would try applying pr-str over a huge (or infiinite - I don't know) object and OOM accordingly.

I'd be inclined to remove the println and simply let users report issues.

A println can be dangerous otherwise and most users typically cannot make much use of it anyway.

Avoid string/includes?

Would break the clojure 1.7 job.