joshuaclayton/unused

Unused is unaware of `alias_method_chain`

sgrif opened this issue · 1 comments

sgrif commented

While it's now deprecated, it's still somewhat common in older codebases. Seems like it'd make sense for any token matching (.*)_with_(.*) to search for alias_method_chain :\1, :\2 as well

@sgrif this is actually a lot trickier to handle than the current alias mechanism, since it only matches one wildcard. I'd based it on the syntax of projectionist in order to do transformations, so there's no way to do multiple capture groups.

Especially given it's deprecated, I'm going to close this for now. If there are a handful of other multi-wildcard aliases that crop up, though, I may have to come up with a way to address this.