Skip to content

CI: check OSMC-PL 1.8 license header in Julia src files#91

Merged
JKRT merged 1 commit into
OpenModelica:masterfrom
SVAGEN26:add-julia-license-check
Jun 25, 2026
Merged

CI: check OSMC-PL 1.8 license header in Julia src files#91
JKRT merged 1 commit into
OpenModelica:masterfrom
SVAGEN26:add-julia-license-check

Conversation

@SVAGEN26

Copy link
Copy Markdown
Contributor

Summary

Adds a CI check that every .jl file under src/ carries the OSMC-PL 1.8 license header with an up-to-date copyright year. This mirrors OpenModelica's check_runtime_license.py, adapted for Julia #= ... =# block comments.

What's included

  • .CI/scripts/check_julia_license.py — the checker. Supports --fix-license, --update-year, --summary, and a .CI/scripts/julia-license-exceptions.txt exclusion list with .gitignore-style (last-match-wins) semantics.
  • .github/workflows/check-license.yml — runs python3 .CI/scripts/check_julia_license.py --summary src on every pull request.
  • Header normalization — brought the src/ files up to the OSMC-PL 1.8 header so the check passes on this PR.

Notes

The check only looks at src/ (not examples/tests). Headers are verified for presence of the OSMC-PL 1.8 text, the "This file is part of OpenModelica." mark, and a current copyright year.

🤖 Generated with Claude Code

Add a CI check that every `.jl` file under `src/` carries the OSMC-PL 1.8
license header with an up-to-date copyright year, mirroring OpenModelica's
`check_runtime_license.py` (adapted for Julia `#= ... =#` block comments).

- `.CI/scripts/check_julia_license.py`: the checker (supports --fix-license,
  --update-year, --summary and a `.CI/scripts/julia-license-exceptions.txt`
  exclusion list with .gitignore-style semantics).
- `.github/workflows/check-license.yml`: runs it on `src` for pull requests.
- Brought the few header-less src files up to the OSMC-PL 1.8 header so the
  check passes.

Co-Authored-By: JKRT <jtinnerholm@gmail.com>
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@codecov

codecov Bot commented Jun 24, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 30.98%. Comparing base (7631d88) to head (2cc5bf0).
⚠️ Report is 19 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff             @@
##           master      #91       +/-   ##
===========================================
+ Coverage   20.40%   30.98%   +10.57%     
===========================================
  Files         166      181       +15     
  Lines       60213    61390     +1177     
===========================================
+ Hits        12288    19022     +6734     
+ Misses      47925    42368     -5557     

☔ 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.

@JKRT JKRT merged commit a36f760 into OpenModelica:master Jun 25, 2026
8 checks passed
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.

2 participants