Skip to content

fix: add missing cspell words and drop phantom analyses path - #292

Open
mrbobbytables wants to merge 1 commit into
mainfrom
fix/spelling-and-phantom-analyses-path
Open

mrbobbytables wants to merge 1 commit into
mainfrom
fix/spelling-and-phantom-analyses-path

Conversation

@mrbobbytables

Copy link
Copy Markdown
Member

What

Fixes two of the three issues from #264 that make npm run check / npm test fail on a clean checkout:

  • check:spelling fails on unmodified main. .cspell.yml was missing six words that already appear in the docs: cutover, retag, ACMM, linuxfoundation, and reprioritizations. Added them to the words list.
  • Scripts reference a phantom analyses/ directory. check:spelling passed analyses as a path arg, and _check:links-md ran find analyses -name '*.md'. That directory has never existed in this repo (git log --all -- analyses is empty). Removed both references.

The Makefile/check:links issue (item 1 in #264) is already covered by #291, so it's left untouched here to avoid conflicting with that PR.

Testing

  • npm run check:spelling now passes (0 issues, 38 files checked).
  • npm run test:unit passes (55/55).

— hive: backend=copilot model=claude-sonnet-5

🐝 Hive Agent: contributor | SHA: e3b18fb

- Add cutover, retag, ACMM, linuxfoundation, and reprioritizations to
  .cspell.yml so check:spelling passes on a clean checkout.
- Drop the nonexistent analyses/ path from check:spelling and
  _check:links-md; the directory has never existed in this repo.

Refs #264

Signed-off-by: copilot-swe-agent <223556219+Copilot@users.noreply.github.com>
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