Skip to content

Commit 6b75c13

Browse files
authored
Merge pull request #87 from dimitribouniol/patch-3
Update `license_header_check_project_name` to no longer be required
2 parents 35529ee + 2ad4ac1 commit 6b75c13

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

.github/workflows/scripts/check-license-header.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,12 +17,11 @@ log() { printf -- "** %s\n" "$*" >&2; }
1717
error() { printf -- "** ERROR: %s\n" "$*" >&2; }
1818
fatal() { error "$@"; exit 1; }
1919

20-
test -n "${PROJECT_NAME:-}" || fatal "PROJECT_NAME unset"
21-
2220
if [ -f .license_header_template ]; then
2321
# allow projects to override the license header template
2422
expected_file_header_template=$(cat .license_header_template)
2523
else
24+
test -n "${PROJECT_NAME:-}" || fatal "PROJECT_NAME unset"
2625
expected_file_header_template="@@===----------------------------------------------------------------------===@@
2726
@@
2827
@@ This source file is part of the ${PROJECT_NAME} open source project

.github/workflows/soundness.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ on:
3737
default: true
3838
license_header_check_project_name:
3939
type: string
40-
description: "Name of the project called out in the license header."
41-
required: true
40+
description: "Name of the project called out in the license header. Required unless `license_header_check_enabled` is false or a `.license_header_template` file is present."
41+
default: ""
4242
broken_symlink_check_enabled:
4343
type: boolean
4444
description: "Boolean to enable the broken symlink check job. Defaults to true."

0 commit comments

Comments
 (0)