search

tsantalis / RefactoringMiner

MIT License
345 stars 132 forks source link

constant change #718

Open wangsiqidahaoren opened 1 month ago

wangsiqidahaoren commented 1 month ago

If an extract method refactoring occurs with an inner constant change. Does RefactoringMiner2.0 considered this as a refactoring operation? @tsantalis

tsantalis commented 1 month ago

@wangsiqidahaoren Do you have a commit where such a change happened? RefactoringMiner could detect such a refactoring, but I am not sure if the extract method detection is running for constant initializers. I need to check this. If you have a commit, it would help a lot.

wangsiqidahaoren commented 3 weeks ago

Sorry I dn't have such a commit, I am just interest of this kind of situation

tsantalis commented 2 weeks ago

@wangsiqidahaoren This is fine. Can you please at least provide an artificial example within the issue with some code, so that I can understand better the change scenario?