Skip to content

cspell: ignore sample/ and test/data/ directories#236

Merged
ilaflott merged 2 commits into
mainfrom
copilot/cspell-workflow
Jun 1, 2026
Merged

cspell: ignore sample/ and test/data/ directories#236
ilaflott merged 2 commits into
mainfrom
copilot/cspell-workflow

Conversation

Copilot AI commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Spell-check was flagging large volumes of false positives from test fixture data — binary pickles, CSVs with usernames/paths, and workload scripts in sample/. These are not source files and don't benefit from spell-checking.

Describe your changes

Added two entries to ignorePaths in .cspell.json:

  • sample/** — workload scripts, staged tarballs, kernel build scripts, migration examples
  • src/epmt/test/data/** — outlier fixtures (outliers/, outliers_nb/), CSV/TSV data, job metadata files

Issue ticket number, link (if applicable)

Checklist

A loose guide to provide structure for contributions

  • the code runs
  • the code is readable
  • the code is commented
  • there are no additional failures in GUARDED pipeline tasks
  • a new test was written (if applicable)
  • new instructions/doc was written (if applicable)
  • I ran pylint and attempted to implement some of it's feedback

Copilot AI linked an issue Jun 1, 2026 that may be closed by this pull request
@ilaflott ilaflott added enhancement New feature or request cleanup labels Jun 1, 2026
Copilot AI changed the title [WIP] Add obvious words to ok-unknown-words list Add cspell spell-check workflow Jun 1, 2026
Copilot AI requested a review from ilaflott June 1, 2026 21:21
Copilot AI changed the title Add cspell spell-check workflow cspell: ignore sample/ and test/data/ directories Jun 1, 2026
@ilaflott ilaflott force-pushed the copilot/cspell-workflow branch from fc9463d to 060e2aa Compare June 1, 2026 22:01
@ilaflott

ilaflott commented Jun 1, 2026

Copy link
Copy Markdown
Member
 Files checked: 130, Issues found: 3016 in 111 files.

also going to be a continue-on-error: true project

@ilaflott ilaflott marked this pull request as ready for review June 1, 2026 22:05
@ilaflott ilaflott merged commit d96b7ea into main Jun 1, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cspell workflow

2 participants