Skip to content
New issue

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

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

Already on GitHub? # to your account

Handle :managed-dependencies and duplicate dependencies #116

Closed
wants to merge 1 commit into from

Conversation

viebel
Copy link
Contributor

@viebel viebel commented Sep 18, 2020

No description provided.

@viebel
Copy link
Contributor Author

viebel commented Sep 18, 2020

Addresses #83 and fixes also upgrade in case of managed dependencies. Without this PR, upgrade would upgrade :dependencies instead of :managed-dependencies

@xsc
Copy link
Owner

xsc commented Dec 21, 2020

Thanks for this! I went a slightly different directions, and instead of removing duplicates only removed those :dependencies that are actually managed: 039bbe5

@xsc xsc closed this Dec 21, 2020
@viebel
Copy link
Contributor Author

viebel commented Dec 21, 2020

Happy to contribute!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants