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

Maven dependencies in the test scope must be ignored #9782

Closed
yegor256 opened this issue Jan 20, 2025 · 1 comment
Closed

Maven dependencies in the test scope must be ignored #9782

yegor256 opened this issue Jan 20, 2025 · 1 comment
Labels
analyzer About the analyzer tool invalid An issue that is deemed to be invalid

Comments

@yegor256
Copy link

yegor256 commented Jan 20, 2025

Maven dependencies in the test scope are not distributed, by the definition of the test scope. Thus, I believe, they must be excluded from the analysis.

@yegor256 yegor256 added bug to triage Issues that need triaging labels Jan 20, 2025
@sschuberth
Copy link
Member

sschuberth commented Jan 20, 2025

Here, in contrast to #9755, I'd disagree: There is no common definition of the meaning of the test scope that guarantees that there's no hypothetical scenario where tests could be distributed.

In general, ORT aims to always get you the complete picture of all dependencies. If some dependencies are not distributed in your scenario, the methodology is to use means like scope excludes to filter ORT's results to match your use-case.

@sschuberth sschuberth added invalid An issue that is deemed to be invalid analyzer About the analyzer tool and removed bug to triage Issues that need triaging labels Jan 20, 2025
@sschuberth sschuberth closed this as not planned Won't fix, can't repro, duplicate, stale Jan 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer About the analyzer tool invalid An issue that is deemed to be invalid
Projects
None yet
Development

No branches or pull requests

2 participants