Skip to content

Fix REUSE compliance#8416

Closed
hilmarf wants to merge 7 commits into
open-telemetry:mainfrom
apeirora:fix/reuse-compliance
Closed

Fix REUSE compliance#8416
hilmarf wants to merge 7 commits into
open-telemetry:mainfrom
apeirora:fix/reuse-compliance

Conversation

@hilmarf

@hilmarf hilmarf commented May 19, 2026

Copy link
Copy Markdown

Summary

Fixes all issues reported by the REUSE API to make the repository REUSE-compliant.

Changes

  • LICENSES/Apache-2.0.txt: Added the Apache-2.0 license file in the location required by REUSE spec (copy of the existing LICENSE file).
  • .reuse/dep5: Added a machine-readable copyright file (Debian format) that covers all files which cannot or should not have inline SPDX headers (config files, build scripts, docs, generated API diffs, resources, etc.).
  • OtelImplJavadocTest.java / OtelInternalJavadocTest.java: Fixed invalid SPDX expressions. These test files embed Java source code as string literals for use with CompilationTestHelper. Each test fixture starts with /* ... SPDX-License-Identifier: Apache-2.0 ... */ — the REUSE linter was picking up Apache-2.0", (the Java string with trailing comma) as an invalid SPDX expression. Added // REUSE-IgnoreStart / // REUSE-IgnoreEnd comments around the embedded license headers to suppress REUSE linter processing of those test fixture strings.

Test plan

hilmarf added 7 commits April 28, 2026 11:43
Signed-off-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
Signed-off-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
Signed-off-by: Hilmar Falkenberg <hilmar.falkenberg@sap.com>
…pressions

- Add LICENSES/Apache-2.0.txt (copy of LICENSE) as required by REUSE spec
- Add .reuse/dep5 to provide copyright/license info for files without inline headers
- Fix invalid SPDX expressions in OtelImplJavadocTest and OtelInternalJavadocTest
  by wrapping embedded SPDX-License-Identifier strings in REUSE-IgnoreStart/End
  comments to prevent the REUSE linter from treating test fixture content as
  actual license declarations
@hilmarf hilmarf requested a review from a team as a code owner May 19, 2026 15:32
@hilmarf hilmarf closed this May 19, 2026
@codecov

codecov Bot commented May 19, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.15%. Comparing base (b29f3df) to head (8d0f1bf).
⚠️ Report is 31 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #8416      +/-   ##
============================================
+ Coverage     90.85%   91.15%   +0.29%     
+ Complexity     7954     7762     -192     
============================================
  Files           896      881      -15     
  Lines         24010    23408     -602     
  Branches       2390     2330      -60     
============================================
- Hits          21815    21337     -478     
+ Misses         1457     1377      -80     
+ Partials        738      694      -44     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hilmarf

hilmarf commented May 19, 2026

Copy link
Copy Markdown
Author

sorry - wrong destination

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant