feat(ci): check licenses on PRs #5636

gansheer · 2024-06-14T09:47:03Z

Add a new licenses github workflow (existing validate workflow has different file list filter)
Fix the check_licenses.sh script to ensure it result in exit 1 when an invalid file is detected
Fix some still missing licenses

Release Note

feat(ci): check licenses on PRs

.github/workflows/licenses.yaml

.github/workflows/validate.yml

gansheer force-pushed the feat/5631_ci_check_licenses branch 7 times, most recently from eb1ed5c to dd3e651 Compare June 14, 2024 12:33

gansheer changed the title ~~[DO NOT MERGE] check version~~ feat(ci): check licenses on PRs Jun 14, 2024

gansheer marked this pull request as ready for review June 14, 2024 12:41

gansheer force-pushed the feat/5631_ci_check_licenses branch from dccddbf to e17c48a Compare June 17, 2024 07:25

gansheer requested review from christophd and squakez June 18, 2024 07:34

squakez reviewed Jun 18, 2024

View reviewed changes

.github/workflows/licenses.yaml Outdated Show resolved Hide resolved

gansheer force-pushed the feat/5631_ci_check_licenses branch from e17c48a to 51babb1 Compare June 18, 2024 08:48

gansheer added 3 commits June 18, 2024 11:05

feat(ci): check licenses on PRs

da91c0d

fix(ci): Fix check licence exit value

69f0e65

chore(ci): Add missing licenses

02b6f29

gansheer force-pushed the feat/5631_ci_check_licenses branch from 51babb1 to 02b6f29 Compare June 18, 2024 09:05

squakez reviewed Jun 18, 2024

View reviewed changes

.github/workflows/validate.yml Show resolved Hide resolved

squakez approved these changes Jun 18, 2024

View reviewed changes

gansheer merged commit 87676e0 into apache:main Jun 18, 2024
12 checks passed

Provide feedback