neiam /home-assistant-plasmoid

Home Assistant Plasmoid

public · Issues · Pulls · Labels · Forks · Compare · Actions success · Badges · Packages

Validate: drop qmllint, which can't resolve anything on the runner

71fd7ea · gmorell · 2026-11-05 22:36

✓ Validate Plasmoid / validate (push) ✓ package / package (push)

3 files +4 -4

Files changed

modified .github/workflows/validate.yml

+4 −4

@@ -82,9 +82,10 @@ jobs:

82	82	qtbin=/usr/lib/qt6/bin
83	83
84	84	# qmlformat parses without resolving imports, so it fails on a
85		− # genuine syntax error and nothing else — that's the gate.
86		− # qmllint goes further but can't see the Plasma/Kirigami modules
87		− # on this runner, so its findings are printed, not enforced.
	85	+ # genuine syntax error and nothing else. Not qmllint: with no QML
	86	+ # modules on the runner — not even QtQuick — it flags every Item
	87	+ # and Rectangle as unresolved, and the 10k lines of that bury the
	88	+ # log past the forge's cap. Run it locally, where Plasma is.
88	89	failed=0
89	90	while read -r qml_file; do
90	91	echo "Checking: $qml_file"

@@ -92,7 +93,6 @@ jobs:

92	93	echo "❌ QML syntax error in: $qml_file"
93	94	failed=1
94	95	fi
95		− "$qtbin/qmllint" "$qml_file" \|\| true
96	96	done < <(find . -name ".qml" -type f -not -path './.git/')
97	97
98	98	if [[ $failed -ne 0 ]]; then

Parents: cfb03ed