Skip to content

Correct embedded documentation typos in kernelSpecs.json#4086

Merged
dkachuma merged 2 commits into
dkachuma/refactor/auto-generate-pvt-driver-kernelsfrom
copilot/fix-code-for-review-comment
Jun 17, 2026
Merged

Correct embedded documentation typos in kernelSpecs.json#4086
dkachuma merged 2 commits into
dkachuma/refactor/auto-generate-pvt-driver-kernelsfrom
copilot/fix-code-for-review-comment

Conversation

Copilot AI commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

This PR addresses a single review comment on PR #3687 requesting exact typo fixes in user-facing embedded documentation strings. The change is limited to the suggested wording updates in kernelSpecs.json.

  • Scope

    • Updated only documentation text under doc.combinations.vars and doc.explicit.
    • No functional/configuration logic changes.
  • Applied review suggestion (exact wording fixes)

    • combinatorallycombinatorially
    • concatingconcatenating
    • can by modifiedcan be modified
  • Patch excerpt

    "vars" : [ "each value in this list will be used to generate kernel template files",
               "combinatorially with all other value lists" ],
    "explicit" : [ "specify a specific combination of the variables to generate",
                   "specify by choosing one value from each variable you want generated",
                   "and concatenating them with '#' (this can be modified internally at the",
                   "generation call). Duplicate entries are unified internally" ]

Copilot AI changed the title [WIP] Fix code as per review comment Correct embedded documentation typos in kernelSpecs.json Jun 17, 2026
Copilot AI requested a review from dkachuma June 17, 2026 01:19
@dkachuma dkachuma marked this pull request as ready for review June 17, 2026 01:21
@dkachuma dkachuma merged commit 62cb10e into dkachuma/refactor/auto-generate-pvt-driver-kernels Jun 17, 2026
1 check was pending
@dkachuma dkachuma deleted the copilot/fix-code-for-review-comment branch June 17, 2026 01:21
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.

2 participants