Chao Li 082b563fc4 .python/SpellCheck: Add "Loongson" and "LOONGARCH" to "words" section 2 lat temu
..
Readme.md 9da7846c88 .pytool/Plugin: Add CI plugins 4 lat temu
SpellCheck.py 2aac8bb7ef .pytool: Update to newest pytools 2 lat temu
SpellCheck_plug_in.yaml 9da7846c88 .pytool/Plugin: Add CI plugins 4 lat temu
cspell.base.yaml 082b563fc4 .python/SpellCheck: Add "Loongson" and "LOONGARCH" to "words" section 1 rok temu

Readme.md

Spell Check Plugin

This CiBuildPlugin scans all the files in a given package and checks for spelling errors.

This plugin requires NodeJs and cspell. If the plugin doesn’t find its required tools then it will mark the test as skipped.

Configuration

The plugin has a few configuration options to support the UEFI codebase.

  "SpellCheck": {
      "AuditOnly": False,          # If True, log all errors and then mark as skipped
      "IgnoreFiles": [],           # use gitignore syntax to ignore errors in matching files
      "ExtendWords": [],           # words to extend to the dictionary for this package
      "IgnoreStandardPaths": [],   # Standard Plugin defined paths that should be ignore
      "AdditionalIncludePaths": [] # Additional paths to spell check (wildcards supported)
  }

AuditOnly

Boolean - Default is False. If True run the test in an Audit only mode which will log all errors but instead of failing the build it will set the test as skipped. This allows visibility into the failures without breaking the build.

IgnoreFiles

This supports .gitignore file and folder matching strings including wildcards

  • All files will be parsed regardless but then any spelling errors found within ignored files will not be reported as an error.
  • Errors in ignored files will still be output to the test results as informational comments.

ExtendWords

This list allows words to be added to the dictionary for the spell checker when this package is tested. These follow the rules of the cspell config words field.

IgnoreStandardPaths

This plugin by default will check the below standard paths. If the package would like to ignore any of them list that here.

[
# C source
"*.c",
"*.h",

# Assembly files
"*.nasm",
"*.asm",
"*.masm",
"*.s",

# ACPI source language
"*.asl",

# Edk2 build files
"*.dsc", "*.dec", "*.fdf", "*.inf",

# Documentation files
"*.md", "*.txt"
]

AdditionalIncludePaths

If the package would to add additional path patterns to be included in spellchecking they can be defined here.

Other configuration

In the cspell.base.json there are numerous other settings configured. There is no support to override these on a per package basis but future features could make this available. One interesting configuration option is minWordLength. Currently it is set to 5 which means all 2,3, and 4 letter words will be ignored. This helps minimize the number of technical acronyms, register names, and other UEFI specific values that must be ignored.

False positives

The cspell dictionary is not perfect and there are cases where technical words or acronyms are not found in the dictionary. There are three ways to resolve false positives and the choice for which method should be based on how broadly the word should be accepted.

CSpell Base Config file

If the change should apply to all UEFI code and documentation then it should be added to the base config file words section. The base config file is adjacent to this file and titled cspell.base.json. This is a list of accepted words for all spell checking operations on all packages.

Package Config

In the package *.ci.yaml file there is a SpellCheck config section. This section allows files to be ignored as well as words that should be considered valid for all files within this package. Add the desired words to the «ExtendedWords» member.

In-line File

CSpell supports numerous methods to annotate your files to ignore words, sections, etc. This can be found in CSpell documentation. Suggestion here is to use a c-style comment at the top of the file to add words that should be ignored just for this file. Obviously this has the highest maintenance cost so it should only be used for file unique words.

// spell-checker:ignore unenroll, word2, word3

or

# spell-checker:ignore unenroll, word2, word3