Skip to content

Fix SpellCheck failure for drill-run test placeholder tokens - #3321

Draft
dynamicdhx with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-spellcheck-job
Draft

Fix SpellCheck failure for drill-run test placeholder tokens#3321
dynamicdhx with Copilot wants to merge 2 commits into
mainfrom
copilot/fix-spellcheck-job

Conversation

Copilot AI commented Aug 19, 2026

Copy link
Copy Markdown
Contributor

The SpellCheck workflow failed because newly introduced drill-run placeholder tokens in Resilience Management tests were not recognized by cspell. This PR updates the repo dictionary so those placeholders are treated as valid terms.

  • Root cause

    • cspell flagged uppercase placeholder tokens used by drill-run tests:
      • DRILLRUNNAME
      • DRILLRUNRESOURCENAME
  • Change applied

    • Updated /home/runner/work/mcp/mcp/.vscode/cspell.json to include lowercase dictionary entries:
      • drillrunname
      • drillrunresourcename
  • Scope

    • Dictionary-only change; no command/service/test logic modifications.
{
  "words": [
    "drawcord",
    "drillrunname",
    "drillrunresourcename",
    "dssldrf"
  ]
}

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Co-authored-by: dynamicdhx <310516170+dynamicdhx@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix failing GitHub Actions job 'SpellCheck' Fix SpellCheck failure for drill-run test placeholder tokens Aug 19, 2026
Copilot AI requested a review from dynamicdhx August 19, 2026 08:14
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