[qt/qt/qtqa]: Summary of bulk changes made
KDE Git Services - Bulk Change <[email protected]>
| Newsgroups | gmane.comp.kde.cvs |
|---|---|
| Message-ID | <[email protected]> |
Git repository change summary for qt/qt/qtqa Pushed by mirror-service into branch 'dev'. Changed from 4e5a7433aab373f27467896f7b6e8842f434363a to 0e30664bbfd034a73a6c08230a46b2cba30d3d70 Acknowledgement was received that this change introduces only existing code that has been pushed to another public open source repository. This change contains the following new commits: Git commit dd0b93aa1d61c4d167023ad8a47d3bcf5551efb1 by Venugopal Shivashankar on 06/08/2026 at 09:29.. vale: restrict \qml and \code BlockIgnores to real code blocks The \qml and \code patterns in BlockIgnores had no delimiter after the command name, so they also matched every command that starts with those letters: \qmltype, \qmlmodule, and 15 other \qml* commands, plus \codeline. A match starting at \qmltype ran lazily to the next \endqml, blanking every prose line in between, so no rule ever saw it. Add \s after each command name. \s matches the newline that normally follows \qml and \code, and also the space in argument forms such as \code [cmake], so real blocks keep being ignored. Also add nativetype, inqmlmodule, since, qtcmakepackage, and qtvariable to the TokenIgnores command alternation. Without them, their arguments reached Pass 1 as prose, so Microsoft.Spacing fired on the periods in qualified module names and Qt.Spelling fired on lowercase CMake variable names. Remove the dead (?s)\\snippet.*?\\endsnippet pattern from BlockIgnores. \endsnippet is not a QDoc command — it appears nowhere in the QDoc source — so the pattern can never match. \snippet arguments are already blanked by the TokenIgnores alternation. Extend tests/CodeBlocks.qdoc to cover both patterns. Change-Id: I571869e7f621f18e5fb4cb82297be11a46ef7ed8 Reviewed-by: Jerome Pasion <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/dd0b93aa1d61c4d167023ad8a47d3bcf5551efb1 Git commit 7e7b5126c9b0fff98e821b8897277eefd0a675aa by Venugopal Shivashankar on 06/08/2026 at 09:29.. vale: suppress Microsoft.Spacing false positives on Qt module names Add a TokenIgnores pattern for qualified Qt module names of the form Qt[Module](.Component)+ so that names like QtQuick.Layouts, QtQml.Models, Qt.labs.qmlmodels, and QtQuick3D.Effects are blanked before Microsoft.Spacing processes the text. The rule's pattern [a-z][.?!][A-Z] would otherwise fire on the period in these names, for example 'l.M' in QtQml.Models. No regex can separate the two cases: a qualified identifier and a missing-space sentence boundary are lexically identical, so this entry also blanks genuine violations that follow a Qt module name. Enumerate the allowed QtQuick and QtQml module name patterns. An enumerated list keeps the blanked surface as small as possible and avoids swallowing a real sentence boundary after an invented name like QtQuick.Then. Anything the pattern blanks is invisible to every enabled rule, including Qt.Spelling. Add test cases to MicrosoftRules.qdoc covering the full range of QtQuick and QtQml module name patterns, including multi-level names (QtQuick.Controls.Material, QtQuick.VirtualKeyboard.Settings) and Qt.labs.* experimental modules. A regression case verifies that a sentence boundary next to a module name still fires, and a SHOULD WARN block verifies the spacing rule fires on genuine violations. Change-Id: Icd09988249e3a1a5c52a62c16662c1a986d1098d Reviewed-by: Alexei Cazacov <[email protected]> Reviewed-by: Jerome Pasion <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/7e7b5126c9b0fff98e821b8897277eefd0a675aa Git commit f50ff78844ea3b62c56d8735e20a1cb59aae29b2 by Venugopal Shivashankar on 06/08/2026 at 09:30.. vale: add SharedContent.qdocinc test fixture for qdocinc linting Exercises lintQDocInc with a multi-section .qdocinc file covering: - \note with passive voice (Microsoft.Passive) - 4-row \table with \c{}, \l{}{}, \macos macro, and passive voice in cells and trailing \note (Microsoft.Passive, Microsoft.Wordiness) - \brief without terminal period (Qt.QDocBrief) - \section1/\section2 with title-case headings (Microsoft.Headings) - Sentence-case headings and clean prose that produce no alerts Each section is delimited by //! [identifier] markers so it can be included selectively via \include SharedContent.qdocinc <id>. Verified against vale v3.15.1-qdoc, which lints .qdocinc prose directly without requiring /*! ... */ doc-comment wrappers. Earlier builds (v3.14.2-qdocsupport-beta1) required the wrappers; this fixture relies on the behaviour introduced in v3.15.1-qdoc. Change-Id: I615b568c938b5dc4cc4d62f06045e1f19b7e4b8b Reviewed-by: Jerome Pasion <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/f50ff78844ea3b62c56d8735e20a1cb59aae29b2 Git commit 638130b71747bbb7bc0162f74837f0d807fcf45a by Venugopal Shivashankar on 06/08/2026 at 09:30.. vale: rename README.txt to README.md and update binary reference Convert the plain-text README to Markdown for better rendering on Gerrit and code hosting platforms. Update the QDoc binary download instruction to name v3.15.1-qdoc as the minimum required version and link to the releases page rather than a pinned tag. The previously pinned build (v3.14.1-qdocsupport-alpha, 2026-03-25) does not lint bare .qdocinc markup; anyone following the old instruction got a linter that silently skipped that file type. Also fix two minor inaccuracies carried over from the original: - styles\Vocab corrected to styles/config/vocabularies/ - "than several" corrected to "rather than several" Change-Id: I7a2b3c4d5e6f7a8b9c0d1e2f3a4b5c6d7e8f9a0b Reviewed-by: Jerome Pasion <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/638130b71747bbb7bc0162f74837f0d807fcf45a Git commit ec2aa1076f77ff5cbcdf92130abd1f6122da27e2 by Jerome Pasion on 06/08/2026 at 11:13.. documentation_review: Add artifact-tester agent and skill artifact-tester checks skills and agents for correctness and suitability in the Qt Documentation Review system. "artifact" is either skill or agent (or both) 9 layers of verification: 1. Define the contract - check the artifact context 2. Static review - check for typos, correctness 3. System integration - how well it fits into the system 4. Trigger test - if and how well the artifact is triggered 5. Dynamic run - run full tests on several prompts 6. Verification & citation - verify the output and cite sources 7. Edge cases - test the confidence and failures 8. Baseline comparison - does the agent/skill improve the system 9. Capture as fixture - generate reports and CHANGELOG entries Register the agent in README.md and agents/agent-comparison.md so it appears in the dispatcher tables alongside the other agents. Task-number: QTBUG-144508 Pick-to: 6.12 Change-Id: I0670b04d5da2724091f7c7cb67126df6b46081e9 Reviewed-by: Venugopal Shivashankar <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/ec2aa1076f77ff5cbcdf92130abd1f6122da27e2 Git commit 8b801c5717c4a4cb5c5b9521e01211de9d66b862 by Jerome Pasion on 06/08/2026 at 11:13.. documentation_review: Restructure skill-language-style for efficiency SKILL.md had grown to roughly 14k tokens, about 2.8x the 5k per-skill Split the monolith into a ~4k-token core plus eight on-demand reference files. SKILL.md goes from 1787 to 426 lines. Pick-to: 6.12 Change-Id: Iff3adb64917a73c88911e9023f021060864c05a3 Reviewed-by: Venugopal Shivashankar <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/8b801c5717c4a4cb5c5b9521e01211de9d66b862 Git commit 0e30664bbfd034a73a6c08230a46b2cba30d3d70 by Jerome Pasion on 06/08/2026 at 11:13.. documentation_review: Update rules across skills and the reviewer This commit groups four independent rule updates. Genus mismatch detection (skill-linking-check, skill-qdoc): Alt text common review corrections (skill-alttext): Qt-specific spellings (skill-language-style): Present-tense and conciseness scans (qt-doc-reviewer, CLAUDE.md): Pick-to: 6.12 Change-Id: I854521a44aa2fdda6763a28209a06aab73f9ef2f Reviewed-by: Venugopal Shivashankar <[email protected]> https://invent.kde.org/qt/qt/qtqa/-/commit/0e30664bbfd034a73a6c08230a46b2cba30d3d70