zk icon indicating copy to clipboard operation
zk copied to clipboard

Make it explicit that resources is initialized as nil

Open jgarte opened this issue 2 years ago • 1 comments

WDYT

jgarte avatar Sep 24 '23 19:09 jgarte

nil is implied here, and I don't see a good reason to make it explicit. did you run into a bug because of this?

also, there are a lot of these let-bound variables in the code, so if one gets nil, they all should. but that just seems unnecessarily verbose to me.

localauthor avatar Sep 25 '23 05:09 localauthor