Skip to content

ci: Make author-lint domain-based and license-agnostic - #169

Merged
DanielKellerM merged 1 commit into
pulp-platform:develfrom
DanielKellerM:ci/flexible-author-lint
Aug 10, 2026
Merged

ci: Make author-lint domain-based and license-agnostic#169
DanielKellerM merged 1 commit into
pulp-platform:develfrom
DanielKellerM:ci/flexible-author-lint

Conversation

@DanielKellerM

Copy link
Copy Markdown
Collaborator

Check attribution only: any copyright holder, any SPDX license line (value stays gated by lint-license), and authors by email domain. Unblocks the Mosaic-copyright files in #161. Verified: devel tree clean, req_replay.sv passes.

Copilot AI lite review requested due to automatic review settings August 10, 2026 10:26

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@DanielKellerM
DanielKellerM force-pushed the ci/flexible-author-lint branch from 2861efa to c4765c0 Compare August 10, 2026 13:03
lint-license and lint-author both hardcoded the copyright holder (ETH
Zurich and University of Bologna / lowRISC), rejecting valid SHL-0.51
contributions from other holders such as Mosaic SoC AG (pulp-platform#161).

Generalize the holder to any org in both checks while keeping the
open-source gate: lint-license still requires the SHL-0.51/Apache-2.0
license text and SPDX, and lint-author now also requires an
SHL-0.51/Apache-2.0 SPDX line (so it still covers the extensions
lint-license skips). Match authors by email domain instead of a
per-person allowlist, and harden the email parse (single @, lowercased
domain, no empty labels).
@DanielKellerM
DanielKellerM force-pushed the ci/flexible-author-lint branch from c4765c0 to 40a7ac5 Compare August 10, 2026 13:04
@DanielKellerM
DanielKellerM merged commit b9a5177 into pulp-platform:devel Aug 10, 2026
12 checks passed
@DanielKellerM
DanielKellerM deleted the ci/flexible-author-lint branch August 10, 2026 13:09
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