Fork me on GitHub
#eastwood
<
2022-09-14
>
andy.fingerhut15:09:32

I thought there was already a linter some years ago that could warn about uses of assoc! and other transient operations, if you called them but discarded their return value. It has been years since I looked at it, but I recall at least considering implementing something like that. Perhaps it bit-rotted since then.

andy.fingerhut15:09:29

Perhaps the :unused-ret-vals linter did that at some point in the past?

thumbnail15:09:54

That’s actually how this was implemented 😁

vemv18:09:55

@U0CMVHBL2 there's a fair chance that assoc! once was defined like that. var-info.edn has had its churn/'bugs'. Now with unit tests it's more difficult that the intent gets lost in the way

andy.fingerhut19:09:57

In any case, cool that you added the capability back in if it was not there in the latest version of Eastwood.

🍻 1