Skip to content

Add license validator for HACS action#5219

Open
Copilot wants to merge 3 commits intomainfrom
copilot/extend-hacs-action-license-validation
Open

Add license validator for HACS action#5219
Copilot wants to merge 3 commits intomainfrom
copilot/extend-hacs-action-license-validation

Conversation

Copy link
Copy Markdown

Copilot AI commented Apr 14, 2026

Extend the action validation to include a check that ensures the repository has a valid license.

Adds a new validator that checks repositories have a valid, SPDX-recognized
license. Repositories with no license or an unrecognized license (key: "other")
will fail validation. Forks are skipped (allow_fork = False).

Agent-Logs-Url: https://github.com/hacs/integration/sessions/12156256-5db1-4375-a92c-a55c0bc68168

Co-authored-by: ludeeus <15093472+ludeeus@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 14, 2026 08:34
@ludeeus ludeeus added the pr: action Changes to actions label Apr 14, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a new repository validator to the HACS GitHub Action that enforces presence of a GitHub-detected, recognized license, along with unit tests covering the expected pass/fail cases.

Changes:

  • Introduces custom_components/hacs/validate/license.py validator that fails when the GitHub API reports no license or an unrecognized license (key == "other").
  • Adds tests for the validator covering: missing license, unrecognized license, and a valid SPDX license.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
custom_components/hacs/validate/license.py New action validator that checks repository_object.attributes["license"] and fails for missing/unrecognized licenses.
tests/validate/test_repository_license_check.py New tests validating failure/success behavior for the new license validator.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ludeeus ludeeus marked this pull request as ready for review April 14, 2026 08:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: action Changes to actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants