search

krdln / shelly

Shelly — very dumb PowerShell script analyzer
MIT License
3 stars 1 forks source link

Teach shelly about classes #20

Closed krdln closed 6 years ago

krdln commented 6 years ago

At least a little bit.

cc @m-kostrzewa

It will enable us to get rid of all but one [Shelly-Bug] in winci (the one remaining is actually a different one—unused-imports is firing because the only usage of function is through Get-Item function:Something).

Interestingly, this PR causes indirect-imports to fire a lot for—as you might guessed—indirectly imported classes.

krdln commented 6 years ago

Still need to do something with this heuristic:

// Temporarily silence unused-imports for files with no functions
// to avoid false positives.