Skip to content

Refactoring on Famix Simple Diff - #63

Merged
RomainDeg merged 34 commits into
moosetechnology:mainfrom
AntoninGoslin:fixing_diff
Jul 24, 2026
Merged

Refactoring on Famix Simple Diff#63
RomainDeg merged 34 commits into
moosetechnology:mainfrom
AntoninGoslin:fixing_diff

Conversation

@AntoninGoslin

Copy link
Copy Markdown
Contributor

This PR mainly brings refactoring and cleaner code with her. Here is the list of the main ideas:

  • Changed the printOn method. Each subclass defines how the printOn should be implemented; for example, additions only show what was added, deletions show what was deleted, etc.
  • Renaming methods like FamixSimpleDifferenceEntityAddition into FSDEntityAddition.
  • I added an exception message when the user tries to reach the list of methods when it is not created.
  • Adding a new class to define the properties related to changes.
  • Refactoring the way we are building the differences. Now every class has its method to build its own difference

@guillep guillep left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some small changes here and there!

Comment thread Famix-Simple-Diff/FamixSimpleDifference.class.st Outdated
Comment thread Famix-Simple-Diff/FSDNullValue.class.st Outdated
Comment thread Famix-Simple-Diff/FSDRelationChanged.class.st Outdated
Comment thread Famix-Simple-Diff/FamixSimpleDiff.class.st Outdated
Comment thread Famix-Simple-Diff/FSDEntityAddition.class.st Outdated
Comment thread Famix-Simple-Diff/FSDEntityChanged.class.st Outdated
Comment thread Famix-Simple-Diff/FSDPropertyChanged.class.st Outdated
Comment thread Famix-Simple-Diff/FSDEntityDeletion.class.st Outdated
@RomainDeg
RomainDeg merged commit 5e3cc7f into moosetechnology:main Jul 24, 2026
1 check passed
@AntoninGoslin
AntoninGoslin deleted the fixing_diff branch July 24, 2026 12:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants