In the spirit of what de-duplication is in Data curation, it seems that my changes were enough to address a basic de-duplication implementation: if method duplicates are found, then first: rename all the invocations of the duplicate methods using the name of the original method declaration. Second: delete the duplicate method declarations from the AST.
This will close Issue #11 and Issue #12
@bdettmer (of course, once you have time), please review the pull request. By reviewing the pull request you would learn how the AST was updated in response to de-duplication calls. Also, you would see in the Pull Request how the CLI was updated to support the De-duplication command.
ghost
commented
10 years ago
In the spirit of what de-duplication is in Data curation, it seems that my changes were enough to address a basic de-duplication implementation: if method duplicates are found, then first: rename all the invocations of the duplicate methods using the name of the original method declaration. Second: delete the duplicate method declarations from the AST.
This will close Issue #11 and Issue #12
@bdettmer (of course, once you have time), please review the pull request. By reviewing the pull request you would learn how the AST was updated in response to de-duplication calls. Also, you would see in the Pull Request how the CLI was updated to support the De-duplication command.