Azure / Azure/azure-libraries-for-java

Maven scope in dependencyManagement

Open
#470 0 comments 0 reactions 0 assignees View on GitHub
customer-reported dependencies Mgmt
Dominant language
Java
Stars
97
Forks
102
PR merge metrics
No merged PRs in 30d

Description

Currently the `commons-io` library is declared with a test scope in the `azure-parent` pom : https://github.com/Azure/azure-libraries-for-java/blob/master/pom.xml#L118

This just caused a rather hard to debug issue for one of my projects where another library used had `commons-io` as a compile dependency (not explicitly set at compile because it is the default), and importing the `azure-parent` "excluded" it, causing my project to fail at runtime.

Quick search shows it may not be a good idea to have scopes in ``, at least not "test" ([first answer I found on SO](https://stackoverflow.com/a/20750041)).

Would it be fair to remove the test scope from `` in `azure-parent`? I can provide the PR.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.