Commit 0dfd55f
authored
Introducing spell checking and fixing several typos (#934)
* fix: introduce cspell and fix spelling errors across codebase
Add cspell spell-checking configuration:
- cspell.json with file/path scoping and Mako template override
- cspell-dict.txt for project-specific technical terms
- cspell-ignore-words.txt for known abbreviations and API identifiers
- cspell-mako-keywords.txt for Mako template keywords
Fix spelling errors in variable, function, file names, comments,
docstrings, and test names throughout the codebase.
* chore: regenerate code after spelling fixes
* fix: address styleguide lint issues
* chore: remove temporary mypy output file
* fix: remove UTF-8 BOM from cspell-ignore-words.txt
* chore: move cspell dictionaries to .cspell/ directory
* chore: merge cspell-ignore-words.txt into cspell-dict.txt
* chore: add DAQmx abbreviations dictionary and deduplicate cspell-dict
* chore: improve cspell configuration and dictionaries
- Set noSuggest on all dictionaries so project-specific terms don't
pollute suggestion lists
- Add cspell-dict and cspell-daqmx-abbreviations to the Mako template
override so abbreviations are recognized in templates
- Merge duplicate entries in cspell-daqmx-abbreviations.txt (e.g.
Conn # Connect / Connected / Connecting / Connector)
- Add '# Real words' section header to cspell-dict.txt
* chore: switch cspell to ignorePaths-only pattern and expand scope to docs
- Replace files allowlist with ignorePaths blocklist and useGitignore: true
- Add docs/**/*.rst to spell-check scope
- Add .cspell/cspell-sphinx-directives.txt dictionary for Sphinx RST directives
- Add *.rst override to apply Sphinx directives dictionary
* chore: expand cspell scope to .md files and fix spelling errors
- Add per-file dictionaries and overrides for CHANGELOG.md and CONTRIBUTING.md
- Add .cspell/cspell-changelog-words.txt with historical identifiers and bot names
- Add .cspell/cspell-contributing-words.txt with contribution workflow terms
- Add package names to cspell-dict.txt (numpy, ctypes, pytest, matplotlib, pyplot)
- Add HWCU and NIDAQ to cspell-dict.txt
- Add bitfields plural form to cspell-dict.txt
- Add backlinks to cspell-sphinx-directives.txt
- Add python dictionary to .rst override for code examples
- fix: correct domaindirectives typo in CHANGELOG.md
* chore: replace blanket extension exclusions with specific file paths in cspell
* chore: sort cspell ignorePaths in VS Code Explorer order
* chore: spell-check .env.sample and add MBCS, multibyte to dictionary
* chore: spell-check .gitignore
* chore: remove redundant comments from cspell overrides
* chore: use consistent dictionary order in cspell overrides
* chore: regenerate utils.py after cspell comment style fix
* chore: run cspell in CI
* docs: document cspell as a required step in the contribution workflow
* chore: consolidate cspell:ignore directives in test_internationalization.py
* chore: remove redundant bnptdms from dictionary
cspell treats \b as a regex metacharacter, so r"\bnptdms\b" produces
the token nptdms, which is already in the dictionary. The bnptdms
entry was never needed.
* refactor: rename FULLSCALE_RAW_MAX/MIN to FULL_SCALE_RAW_MAX/MIN in test_in_stream
* chore: alphabetize cspell dictionary and annotate each entry with its origin
* chore: sort and annotate sphinx directives and mako keywords dictionaries
* chore: sort and annotate changelog and contributing dictionaries, add python dict to changelog override
* chore: migrate changelog and contributing word lists to inline cspell ignores
* chore: extend cspell to codegen/generated files; fix typos
Add .cspell/cspell-codegen-dict.txt with ~115 entries covering
NI-DAQmx initialisms, abbreviations, compound identifiers, typos
locked in public API names, and real domain words specific to
codegen and generated source files.
Update cspell.json:
- Add cspell-codegen-dict dictionary definition
- Remove broad generated/** and src/codegen/** ignore paths so
these files are now spell-checked
- Add generated/nidaqmx/_stubs/** back to ignorePaths (protobuf
stubs are too noisy to check)
- Add override applying cspell-codegen-dict to src/codegen/**
and generated/**
- Sort dictionaryDefinitions alphabetically
- Order ignorePaths to match VSCode Explorer layout
- Add ordering comments to both lists
Fix typos in metadata source (src/codegen/metadata/attributes.py):
- specfied -> specified
- thermcouple -> thermocouple
- thefilter -> the filter (two occurrences)
These propagate into generated files via codegen; also fix them
directly in the generated files that are already up to date.
Minor code fixes:
- src/codegen/generator.py: docstring "Codegenerator" -> "Code generator"
- src/codegen/stub_generator.py: rename grpc_codegened_file_paths
-> grpc_code_generated_file_paths
* chore: clean up and annotate cspell.json -> ignorePaths
* chore: extend cspell coverage to pyproject.toml
- Remove pyproject.toml from cspell.json ignorePaths
- Add cspell override for pyproject.toml with [cspell-dict,
cspell-daqmx-abbreviations, python] dictionaries; the python
dictionary covers: mypy, pypy, grpcio, tzlocal, doctest,
filterwarnings, xfail
- Add // cspell:disable-line to the override's filename line in
cspell.json to suppress pyproject in that string literal
- Suppress maintainer names (Hindes, Maxx, Boehme, Keryan) using
# cspell:disable / # cspell:enable around the maintainers block
- Add # cspell:ignore addopts above [tool.pytest.ini_options].addopts
- Add # cspell:ignore testpaths above [tool.pytest.ini_options].testpaths
- Add grpcdevice and styleguide to .cspell/cspell-dict.txt
- Remove grpcdevice from .cspell/cspell-changelog-words.txt
- Remove styleguide from .cspell/cspell-changelog-words.txt and
.cspell/cspell-contributing-words.txt
- Remove <!-- cspell:ignore styleguide --> from CONTRIBUTING.md
- Remove <!-- cspell:ignore styleguide --> and
<!-- cspell:ignore grpcdevice --> from CHANGELOG.md
* chore: align ignorePaths comments in cspell.json
* fix: wrap long list comprehension in stub_generator.py
* revert: undo _dotenv_path rename; add dotenvpath to cspell dict with TODO
* refactor: rename cfunc to c_func in _lib.py for consistency with _library_interpreter.py
* chore: remove unused words from cspell-daqmx-abbreviations.txt
* chore: consolidate NI-DAQmx words into cspell-daqmx-abbreviations.txt
* chore: reorganize cspell dictionaries by purpose
* chore: move cspell:ignore comments inline in CONTRIBUTING.md
* chore: delete cspell-changelog-words and cspell-contributing-words dictionaries
* chore: add python to global cspell dictionaries; remove redundant overrides
* fixup! chore: Consolidate cspell dictionaries into one project dictionary
Improve comments in cspell-daqmx-api-elements.txt for consistency:
- Use specific comment taxonomy: abbreviation, unseparated words,
unseparated initialisms, initialism, Typo, Deprecated
- Move mioDAQ to cspell-real-words.txt
- Move forcebridge/pressurebridge/torquebridge to
cspell-project-software-terms.txt (typo: filenames should be PascalCase)
- Move installdriver to cspell-project-software-terms.txt (CLI command)
- Reclassify BBULK, COULDNT, DOESNT, DAQError, notKnown, zidx
with accurate comments instead of 'compatibility breaking name'
* chore: rename TEDS asset files to PascalCase for consistency
forcebridge.ted → ForceBridge.ted
pressurebridge.ted → PressureBridge.ted
torquebridge.ted → TorqueBridge.ted
Update conftest.py fixtures and remove the words from the cspell dictionary.
* chore: improve cspell-real-words.txt comments; correct SMIO expansion
- Remove redundant "Real word:" and "Initialism:" prefixes from comments
- Lowercase entries that are neither initialisms nor proper nouns
- Add descriptions to bare entries
- Standardize GPS initialism comments
- Correct SMIO expansion: Serial Memory I/O -> Simultaneously-sampling Multifunction I/O
* chore: use cspell-action instead of npx to run cspell
* chore: move cspell config to .config/; simplify dictionary names
- Move cspell.json to .config/cspell.json
- Move .cspell/ dictionary files to .config/cspell/, dropping the
redundant "cspell-" prefix from each filename and dictionary name
- Add globRoot: "${cwd}" so paths resolve correctly from the new location
- Set noSuggest: false on real-words to enable spelling suggestions
* docs: add 1.5.0 changelog section; mention cspell under Major Changes
* docs: incorporate 1.4.1 section and fix 1.5.0 PR link in changelog1 parent 257f46e commit 0dfd55f
File tree
56 files changed
+2890
-2548
lines changed- .config
- cspell
- .github/workflows
- examples
- generated/nidaqmx
- system
- task
- triggering
- src
- codegen
- metadata
- templates
- library_interpreter
- task/triggering
- utilities
- handwritten
- tests
- acceptance
- component
- system
- _collections
- storage
- task
- channels
- legacy
- test_assets/teds
Some content is hidden
Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.
56 files changed
+2890
-2548
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
| 104 | + | |
| 105 | + | |
| 106 | + | |
| 107 | + | |
| 108 | + | |
| 109 | + | |
| 110 | + | |
| 111 | + | |
| 112 | + | |
| 113 | + | |
| 114 | + | |
| 115 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
32 | 32 | | |
33 | 33 | | |
34 | 34 | | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
35 | 40 | | |
36 | 41 | | |
37 | 42 | | |
| |||
0 commit comments