Skip to content

(towards #1995) Update headers to conform to SPDX (part2) - #3566

Open
sergisiso wants to merge 5 commits into
masterfrom
1995_headers_part2
Open

(towards #1995) Update headers to conform to SPDX (part2)#3566
sergisiso wants to merge 5 commits into
masterfrom
1995_headers_part2

Conversation

@sergisiso

Copy link
Copy Markdown
Collaborator

No description provided.

@sergisiso
sergisiso requested a review from arporter August 21, 2026 14:11
@sergisiso

Copy link
Copy Markdown
Collaborator Author

Maybe one for you @arporter , this is the same change to the headers but this time to Makefile, x90, and script files.

Importantly, I took a different path for md/readmes. In their case I completely deleted the license and authors. This is because as in the others is not legally required (we have git authoring and a root license), but in this case it can be a minor nuisance for people who read the readmes from a place that doesn't hide comments, and for emedding them in other documentations formats (which I want to start doing more to remove duplications). So I went with the option that I believe has less friction, but let me know if you feel differently.

@codecov

codecov Bot commented Aug 21, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (72527a7) to head (9312e57).

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #3566   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          397       397           
  Lines        55775     55775           
=========================================
  Hits         55775     55775           

☔ View full report in Codecov by Harness.
📢 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.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant