StaticLint.jl icon indicating copy to clipboard operation
StaticLint.jl copied to clipboard

Case of `include(joinpath(...))` with explicit strings

Open theogf opened this issue 4 years ago • 2 comments

RIght now all include of type

include(joinpath("subdir", "myfile.jl"))

are ignored by the linter. Although it's apparently not best practice it would be great for the linter to be able to treat them as include("subdir/myfile.jl").

theogf avatar Sep 13 '21 12:09 theogf

We could offer a code action to remove the unnecessary joinpath :)

davidanthoff avatar Sep 15 '21 20:09 davidanthoff

Our code already tries to handle this case, but it's obviously buggy.

pfitzseb avatar Sep 15 '21 20:09 pfitzseb