Conversation
There was a problem hiding this comment.
Pull request overview
Updates the reusable “apply REUSE headers” workflow to default to applying no license (instead of Apache-2.0) when callers don’t explicitly specify a license, aligning the default behavior with the repository template’s intent.
Changes:
- Changed the
workflow_callinputlicensedefault fromapache-2.0toLicenseRef-NONE.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| description: "The SPDX name of the license to use." | ||
| type: string | ||
| required: false | ||
| default: apache-2.0 | ||
| default: LicenseRef-NONE |
There was a problem hiding this comment.
The input description says this should be an SPDX license identifier, but LicenseRef-NONE is a custom LicenseRef-* (and typically requires a corresponding license text under LICENSES/ for REUSE compliance). If the intent is truly “no license”, consider using an SPDX-supported sentinel like NONE/NOASSERTION (if supported by the underlying reuse tool) or update the workflow/action to support omitting the --license flag entirely; otherwise update the description to indicate a full SPDX license expression (including LicenseRef-*) and ensure downstream REUSE lint won’t fail due to a missing license text.
Updates the default action runner to use no license. This is used by UoMResearchIT/RSE-Repository-Template#77