Skip to content

ci: tolerate version-only differences in THIRD_PARTY_LICENSES.md and the SBOM; exact check at release - #179

Merged
RobbinBouwmeester merged 3 commits into
mainfrom
ci/licence-check-versions
Oct 11, 2026
Merged

RobbinBouwmeester merged 3 commits into
mainfrom
ci/licence-check-versions

Conversation

@RobbinBouwmeester

Copy link
Copy Markdown
Member

Dependabot does not run ci/gen_third_party_licenses.py or ci/gen_sbom.py, so every one of its pull requests failed the "python sidecars + configs" job on a version string in THIRD_PARTY_LICENSES.md (#154, #156, #157, #158, #176), and needed a regeneration commit by hand.

Change

  • ci/_lockfile_versions.py: --check compares the committed and regenerated text with every semantic version masked (x.y.z, with pre-release and build suffixes such as 2.0.16+zstd.1.5.7; two-part numbers like "Version 2.0" and years are left alone). A version-only difference passes with a note that names the regeneration command.
  • It still fails when a crate is added or removed, when a crate appears in a second version (the build(deps): Bump mzdata-bindata from 0.66.6 to 0.67.1 in /rust/mumdia #158 case), and when a licence expression, repository or notice text changes. Those are the parts the licence obligations rest on.
  • --check --strict compares exactly. release.yml (validate-tag) now runs it for both files after cargo fetch --locked, so a tagged release always ships documents that name the exact versions inside the binary. The release PR regenerates both files, as before.
  • tests/python/test_lockfile_versions.py (6 tests); docs/14 and the CHANGELOG updated.

Validation

🤖 Generated with Claude Code

RobbinBouwmeester and others added 3 commits October 11, 2026 10:41
…ents

Dependabot does not run gen_third_party_licenses.py or gen_sbom.py, so every
one of its pull requests failed CI on a version string (#154 #156 #157 #158
#176). --check now compares with semantic versions masked and still fails
on an added or removed crate, a second version of a crate, a licence or a
notice change. --check --strict compares exactly; the release gate runs it.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
check_doc_refs.py reads any *.md string in tracked files as a document citation.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@RobbinBouwmeester
RobbinBouwmeester merged commit 2ee5dc9 into main Oct 11, 2026
12 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.

1 participant