Include spell checker in github ci/cd - #665
Conversation
|
In general, this looks good. We have another solution though at https://github.com/FAIRmat-NFDI/pynxtools/pull/662/files#diff-4dfe935b4852393622c2b414783d1d11e1a868fd8485709124d90208fa61ec8b based on cspell (we are using the same for the Most likely, for any spellchecker, we will need a list of custom words that will not be checked (e.g., NeXus-specific terms). I think the tool used here also supports this. |
|
Thought about this a bit more, I think we should probably use Here's the output of the two on the docs in the master branch: cspell codespell Seems like - name: Spell check paper
uses: streetsidesoftware/cspell-action@v6
with:
files: |
paper/*.md
config: cspell.json
strict: true # Fails if there are any issuesWe can then use the same configuration as in #622. What do you think @RubelMozumder? |
Yeah, I also noticed the |
Indeed, it makes total sense to use |
It's true, for locally checking, one needs |
cspell is implemented! |
Okay, good! But why not use the existing configuration in the docs PR? |
Spell Check for pynxtools Joss paper.