-
Notifications
You must be signed in to change notification settings - Fork 34
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
MergeStrategy with ExtensionGraphAssociation Option for GraphCollector #48
Conversation
- introduce ExtensionGraphAssociation option defining if extension graphs are merged with the extending or extended graph - add unit-test testing ExtensionGraphAssociation behavior
ae26406
to
c6e680f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, thanks! I just have a couple questions.
@swift-ci Please test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, thanks!
@swift-ci Please test |
@swift-ci Please test |
@swift-ci Please test macOS |
This adaption is required to fix apple/swift-docc#210
Summary
Adds a strategy option to
GraphCollector
's init. TheExtensionGraphAssociation
option decides if extension graphs are merged with the extending or extended graph.Dependencies
This PR does not introduce a breaking change and can be merged independently.
Testing
I added a new unit test to test the merging behavior for both ExtensionGraphAssociation options.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.
./bin/test
script and it succeeded