Ján

Should this include new linters? We have #291 and #284 pending, both of which are likely to cause a significant number of new errors in most projects. Also, any opinion...

> there were 0 known usages. I think it's used from hh-apidoc, definition-finder, and/or user-documentation But I agree, we should have tests too.

also the AwaitInLoop linter intentionally overrides the default output to print out specific lines but doesn't point to the exact relevant node at each line

I think I'm missing something; in your example `$apple` is only mentioned once so renaming it to `$_apple` should be fine...? Even if you add something like ```hack $apple =...

Oh of course, somehow I missed that `$apple` was used in the foreach loop, sorry. I agree, we should err on the side of excluding any questionable warnings or autofixes,...

This sounds reasonable to me and probably a simple change (possibly just change `ln -sf ...` to `cp ...` in the build script?) but I don't know a lot about...