-
Notifications
You must be signed in to change notification settings - Fork 308
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
Allow to skip excluded paths (and scopes) during analysis #5968
Comments
Fun fact, we had a |
Hi @sschuberth, that is indeed interesting. And also the comment from @mnonnenmacher states, that it will be reimplemented later. So it was always planned to have this feature back, I guess? I would really appreciate it :) |
Another use case that could motivate this issue: We have a Conan project with the following issue:
In our case, all the Conan files in /tests are excluded in |
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
If the analyzer is configured to skip excludes, it passes the Excludes defined in the repository configuration to PackageManager.findManagedFiles(). That way the analyzer result will contain only projects that are not matched by a path exclude. Resolves oss-review-toolkit#5968. Signed-off-by: Oliver Heger <oliver.heger@bosch.io>
In ORT's semantics "excluded" entities will still get analyzed / scanned by default, but the results from these show up marked as "excluded" in the reports.
This is a problem if you need to really skip e.g. a path from analysis, for example because that path contains an Android or NuGet project that cannot be properly analyzed.
The scanner (and also the advisor) already have a
--skip-excluded
CLI option. The proposal is to introduce--skip-excluded
also to the analyzer, so thatThat way, problematic / large projects that are also irrelevant compliance-wise in the specific project-context can be skipped and completely omitted from the results.
The text was updated successfully, but these errors were encountered: