Skip to content

Apply qmlformat to all QML files - #3

Merged
luukvisser merged 1 commit into
chore/license-readme-toolingfrom
chore/qmlformat
Aug 6, 2026
Merged

luukvisser merged 1 commit into
chore/license-readme-toolingfrom
chore/qmlformat

Conversation

@luukvisser

Copy link
Copy Markdown
Owner

Runs pyside6-qmlformat over all four QML files, so the repository is clean against the qmlformat hook rather than leaving the reformat to whichever future change happens to touch a file first.

Stacked on chore/license-readme-tooling (#2), since the workflow and hook config it checks against live there. Retarget to main once #2 merges.

What changed

Whitespace only:

  • tabs → four spaces
  • single-line anchors { ... } and width: X; height: Y blocks expand onto their own lines
  • single-statement if bodies drop their braces
  • the stray semicolon after import qb.base 1.0 is removed

Verification

No device testing is possible, so the change was verified mechanically:

  • Semantic equivalence: with all whitespace and semicolons stripped, each of the four files is byte-identical to its previous version. Nothing reordered, no property dropped or altered, no comment lost.
  • pyside6-qmllint passes on all four files.
  • The full pre-commit run --all-files suite passes.

Still worth a sanity check on a real Toon before the next release, since none of the above exercises the Toon's own QML runtime.

🤖 Generated with Claude Code

Runs pyside6-qmlformat over the four QML files so the repository is clean
against the qmlformat hook and the PR Lint workflow, instead of leaving the
reformat to whichever change happens to touch a file first.

Whitespace only: tabs become four spaces, single-line anchors/width/height
blocks expand onto their own lines, single-statement if bodies drop their
braces, and the stray semicolon after "import qb.base 1.0" is removed.

Verified no semantic change — with whitespace and semicolons stripped, all
four files are byte-identical to their previous versions. qmllint passes.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@luukvisser
luukvisser merged commit 3bf8929 into chore/license-readme-tooling Aug 6, 2026
2 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