ci: Add ort#93
Merged
Merged
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #93 +/- ##
==========================================
- Coverage 87.87% 87.19% -0.69%
==========================================
Files 21 21
Lines 363 367 +4
==========================================
+ Hits 319 320 +1
- Misses 27 29 +2
- Partials 17 18 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Pre-create output directories and run containers as host user to avoid permission errors writing to mounted volumes. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Run as the container ort user instead of overriding with -u flag, mount cache to /home/ort/.ort, and chmod output dirs so the container user can write to them. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Remove copyleft warning rules as all projects are open source. Detect root license from scan results at script load time instead of using projectSourceGetDetectedLicensesByFilePath in packageRule. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Use full SPDX identifiers (GPL-3.0-only etc.) as preset keys so they match what ORT detects. Move license check from require block (which only accepts RuleMatcher objects) to an if guard. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Add .ort.yml with curations for false positive license detections in CONTRIBUTING.md (GPL-1.0-or-later, NOASSERTION). Use correct SPDX identifiers in LicensePresets and replace generic howToFix messages with actionable guidance. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
timo-reymann
force-pushed
the
ci/add-ort
branch
from
e28a781 to
30ef51e
Compare
The ORT Docker image runs as non-root by default, but GitHub Actions runner mounts temp directories with root-only write permissions. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Checks ClearlyDefined for existing scan results before running scans, reducing scan time for known open source components. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
timo-reymann
force-pushed
the
ci/add-ort
branch
from
b3ef921 to
653a095
Compare
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
License Compliance — ✅ passedI am using ORT to check for license compliance for added / modified library code. This helps adhering to license terms. Reports |
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Relates OR Closes #0000
Description