Skip to content

docs: fix spelling in QML comments, mock params, and guides#14611

Open
Bartok9 wants to merge 4 commits into
mavlink:masterfrom
Bartok9:aerial/comment-typos-2026-07-16
Open

docs: fix spelling in QML comments, mock params, and guides#14611
Bartok9 wants to merge 4 commits into
mavlink:masterfrom
Bartok9:aerial/comment-typos-2026-07-16

Conversation

@Bartok9

@Bartok9 Bartok9 commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Fix vertialvertical and ListVIewListView in QGCFlickable/QGCListView header comments.
  • Fix maxiummaximum in MockLink FW L1 longDesc.
  • Fix duplicated words: Autotune is is, map palette the the.

Motivation

Pure documentation/comment hygiene — easy reviewer win, no runtime behavior change.

Verification

  • Searched fixed strings; gone
  • python3 -m json.tool on MockLink.Parameter.MetaData.json

Notes

AI-assisted; human-reviewed. Docs/comments only.

@codecov

codecov Bot commented Jul 16, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 31.57%. Comparing base (f29efd3) to head (73cc2a3).
⚠️ Report is 156 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #14611      +/-   ##
==========================================
+ Coverage   25.47%   31.57%   +6.10%     
==========================================
  Files         769      785      +16     
  Lines       65912    66985    +1073     
  Branches    30495    31029     +534     
==========================================
+ Hits        16788    21152    +4364     
+ Misses      37285    31454    -5831     
- Partials    11839    14379    +2540     
Flag Coverage Δ
unittests 31.57% <ø> (+6.10%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 455 files with indirect coverage changes


Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5d8a17d...73cc2a3. Read the comment docs.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions

github-actions Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Build Results

Platform Status

Platform Status Details
Linux Passed View
Windows Passed View
MacOS Passed View
Android Passed View

All builds passed.

Pre-commit

Check Status Details
pre-commit Failed (non-blocking) View

Pre-commit hooks: 0 passed, 0 failed, 0 skipped.

Test Results

linux-coverage-integration: 33 passed, 0 skipped
linux-coverage-unit: 107 passed, 0 skipped
Total: 140 passed, 0 skipped

Code Coverage

Coverage: 67.8%

No baseline available for comparison

Artifact Sizes

Artifact Size
QGroundControl 221.83 MB
QGroundControl-aarch64 179.42 MB
QGroundControl-installer-AMD64 137.93 MB
QGroundControl-installer-AMD64-ARM64 80.46 MB
QGroundControl-installer-ARM64 109.19 MB
QGroundControl-linux 87.10 MB
QGroundControl-mac 87.09 MB
QGroundControl-windows 86.16 MB
QGroundControl-x86_64 191.65 MB
No baseline available for comparison

Updated: 2026-07-19 05:11:24 UTC • Commit: 73cc2a3 • Triggered by: Android

@Bartok9
Bartok9 requested a review from HTRamsey as a code owner July 16, 2026 12:23
@github-actions github-actions Bot added the github_actions Pull requests that update GitHub Actions code label Jul 16, 2026
@DonLakeFlyer DonLakeFlyer added this to the Release V5.1 milestone Jul 16, 2026
Comment thread .github/build-config.json
}
},
"ca_bundle_sha256": "86a1f3366afac7c6f8ae9f3c779ac221129328c43f0ab2b8817eb2f362a5025c"
"ca_bundle_sha256": "3ff344e30b9b1ed2971044eabb438a08f2e2245ddb5f8ab1a3ad8b63ab4eaf91"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why is this changed here in a doc pul??

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch — that shouldn't have landed on a docs PR.

The ca_bundle_sha256 pin is only needed for the iOS CI cacert fetch (current curl.se PEM). It belongs on the dedicated PR #14612. I pushed a follow-up that reverts .github/build-config.json on this branch so this PR is spelling-only again. Sorry about the drive-by.

@github-actions github-actions Bot removed the github_actions Pull requests that update GitHub Actions code label Jul 17, 2026
@Bartok9
Bartok9 force-pushed the aerial/comment-typos-2026-07-16 branch from 91e73bd to 7a7dda1 Compare July 17, 2026 20:23
@github-actions github-actions Bot added the github_actions Pull requests that update GitHub Actions code label Jul 17, 2026
@Bartok9
Bartok9 force-pushed the aerial/comment-typos-2026-07-16 branch from 7a7dda1 to bfc8564 Compare July 18, 2026 20:28
Bartok9 added 2 commits July 18, 2026 20:25
Correct vertial/ListVIew headers, maxium in mock FW metadata, and
duplicate words (is is, the the) in en user/dev guide.
Keep the iOS CA pin change on its dedicated CI PR; this branch is spelling-only.
@Bartok9
Bartok9 force-pushed the aerial/comment-typos-2026-07-16 branch from bfc8564 to 0018783 Compare July 19, 2026 00:25
@github-actions github-actions Bot removed MAVLink github_actions Pull requests that update GitHub Actions code labels Jul 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants