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.

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

Remove system collections associations (watch this after "added package tags" pull request because of refactoring) #113

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

AnastasiaKallisto
Copy link

@AnastasiaKallisto AnastasiaKallisto commented Dec 13, 2024

  1. New tag: removeSystemCollectionsAssociations (meh, too long name, but no ideas how to make it short)

How it works:

  • If you will add this tag "-removeSystemCollectionsAssociations", the way, how diagram will be drawn, will be changed. It easier to show, then explain, so...

There is BEFORE:
diagram-16671855984065390642

And AFTER:
diagram-5121057039270088098

P.S.
test file:
image

Program.cs Main

Dictionary<string, string> parameters = MakeParameters(new[]
        {
            "D:\\PlantUmlClassDiagramGenerator\\test\\SourceGeneratorTest\\Planets",
            "D:\\PlantUmlClassDiagramGenerator\\test\\SourceGeneratorTest\\Planets\\PlantUmlDiagrams", 
            "-dir", 
            "-createAssociation", 
            "-allInOne",
            "-removeSystemCollectionsAssociations"
        });

@AnastasiaKallisto AnastasiaKallisto marked this pull request as draft December 13, 2024 09:57
@AnastasiaKallisto AnastasiaKallisto force-pushed the kallisto/feature/removeSystemCollectionsAssociations branch from 195037f to 1f3d98f Compare December 13, 2024 10:08
@AnastasiaKallisto
Copy link
Author

There will be the same work in RecordVisitor and so on

in fields, property, in difficult field and so on - distinct
+ equals and unique relations between classes in RelationshipCollection
@AnastasiaKallisto AnastasiaKallisto force-pushed the kallisto/feature/removeSystemCollectionsAssociations branch from 1f3d98f to f6867ca Compare December 13, 2024 10:47
@AnastasiaKallisto
Copy link
Author

and there will be updates with Action, Func, ILogger and so on

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.

1 participant