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

[MSHADE-318] - Include transitive dependencies #20

Merged
merged 1 commit into from
Jun 21, 2019

Conversation

mkarg
Copy link
Contributor

@mkarg mkarg commented Apr 14, 2019

Fixes MSHADE-318 - Specifically included class's dependencies are missing.

Solution: Include transitive dependencies of specifically included classes.

@mkarg mkarg force-pushed the MSHADE-318 branch 2 times, most recently from 394592f to 8262abe Compare April 15, 2019 20:09
@mkarg mkarg marked this pull request as ready for review May 2, 2019 18:08
@mkarg
Copy link
Contributor Author

mkarg commented May 2, 2019

@khmarbaise Review please. :-)

Includes all transitive dependencies of specifically included classes.

Signed-off-by: Markus KARG <[email protected]>
@mkarg
Copy link
Contributor Author

mkarg commented May 12, 2019

@jvanzyl @olamy @brettporter Review please! :-) This issue is waiting for a review since one month. How are we supposed to work together if all my PRs are unprocessed for weeks?

@mkarg
Copy link
Contributor Author

mkarg commented May 20, 2019

@struberg Review please. :-)

Copy link

@eolivelli eolivelli left a comment

Choose a reason for hiding this comment

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

I took a look but I am not still an expert of this module.
The change looks good to me.

I would like the opinion of another committer before moving forward.

Thank you

@mkarg
Copy link
Contributor Author

mkarg commented May 31, 2019

@rfscholte @khmarbaise Ping ;-)

@rfscholte rfscholte merged commit e1f181f into apache:master Jun 21, 2019
@mkarg mkarg deleted the MSHADE-318 branch June 21, 2019 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants