dgsuarez / reruby

Refactor Ruby
MIT License
35 stars 2 forks source link

Avoid rewriting unrelated requires using 'rename_const' #19

Closed dcarral closed 6 years ago

dcarral commented 6 years ago

Context available in PR https://github.com/dgsuarez/reruby/pull/18, including tests illustrating the current problem.

From there, @dgsuarez wrote:

I think the right way to do this would be to forget about requires as strings and use file path logi:, we want to know if two require paths are the same, or if one is an ancestor of the other. Maybe this stackoverflow question can help us.