Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Combine ClassDiffer, SourceFileDiffer and FunctionDiffer into a single class #113

Open
mkshiblu opened this issue Jun 14, 2021 · 0 comments
Open

Combine ClassDiffer, SourceFileDiffer and FunctionDiffer into a single class #113

mkshiblu opened this issue Jun 14, 2021 · 0 comments
Assignees
@mkshiblu

Comments

@mkshiblu
Copy link
Owner

mkshiblu commented Jun 14, 2021 •
edited
Loading

All of the codes are similar and can be combined in a similar structure, this could also help us allow to implement recursive container matching pattern
@mkshiblu mkshiblu self-assigned this Jun 25, 2021
@mkshiblu mkshiblu added this to the MILESTONE_JUN_30 milestone Jun 25, 2021
@mkshiblu mkshiblu added the In progress Currently doing label Jun 28, 2021
mkshiblu added a commit that referenced this issue Jun 29, 2021
@mkshiblu
#113: Paramterize ContainerDiff to allow merging of SourceFileDiff & …
bc9061b 
…ClassDiff
mkshiblu added a commit that referenced this issue Jul 1, 2021
@mkshiblu
#113: Combined SourceDiff and ContainerDiff
08b7b81
mkshiblu added a commit that referenced this issue Jul 5, 2021
@mkshiblu
#113: Parameterize container diff to allow source and class diff to r…
72f1619 
…e-use the same diff
mkshiblu added a commit that referenced this issue Jul 7, 2021
@mkshiblu
#113: Added rename class checker on model diff
e688694
mkshiblu added a commit that referenced this issue Jul 28, 2021
@mkshiblu
#113: Omit duplicated code in ClassDiffer and ContainerDIffer
c0f1db7
@mkshiblu mkshiblu removed the In progress Currently doing label Sep 16, 2021
@mkshiblu mkshiblu removed this from the MILESTONE_20201_AUG_15 milestone Sep 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mkshiblu mkshiblu

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@mkshiblu