Skip to content

Add MIT license (fix HACS validate failure) - #35

Merged
fboundy merged 1 commit into
mainfrom
patch
Jul 9, 2026
Merged

fboundy merged 1 commit into
mainfrom
patch

Conversation

@fboundy

@fboundy fboundy commented Jul 9, 2026

Copy link
Copy Markdown
Owner

Summary

The scheduled Validate workflow (hacs/action) has been failing daily with 2/9 checks:

  1. <Validation license> failed: The repository has no license — persistent, real: neither this repo nor the upstream fork (Pho3niX90/solis_modbus, also unlicensed) ever declared a license, and GitHub's license metadata is null.
  2. <Validation integration_manifest> failed: expected a dictionary. Got None — transient: preceded by a raw.githubusercontent.com rate-limit warning at the scheduled run time. The raw manifest fetches fine (HTTP 200, valid JSON) outside the throttle window, so this is HACS-action infra flakiness, not a repo defect.

This PR adds an MIT LICENSE (chosen by the maintainer; the HA custom-integration convention). GitHub auto-detects it via Licensee once on main, populating the repo license field and passing the HACS license check.

Notes

  • No code change; license only.
  • The manifest rate-limit flake isn't repo-fixable (the HACS action's own unauthenticated raw fetches get throttled); it should pass once not throttled. Will confirm with a manual workflow_dispatch of Validate after merge.

Fixes the recurring scheduled Validate failure.

The HACS "Validate" workflow fails the license check ("The repository
has no license") because neither this repo nor the upstream fork ever
declared one. Add an MIT LICENSE so GitHub detects it and HACS
validation passes.

Fixes the recurring scheduled Validate failure.
@github-actions

github-actions Bot commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Failed to generate code suggestions for PR

@fboundy
fboundy merged commit 258d770 into main Jul 9, 2026
4 of 6 checks passed
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.

1 participant