ci: add spellchecker to flux-accounting

flux-framework / flux-accounting

bank/accounting interface for the Flux resource manager

https://flux-framework.readthedocs.io/projects/flux-accounting/en/latest/index.html

GNU Lesser General Public License v3.0

3 stars 10 forks source link

ci: add spellchecker to flux-accounting #504

Closed cmoussa1 closed 3 weeks ago

cmoussa1 commented 3 weeks ago

Problem

flux-accounting does not have a spellchecker that runs on pull requests.

This PR adds a spellchecker as another builder during CI. As a result of the spellchecker running (and my horrendous spelling), I've also fixed all of the spelling mistakes flagged by the spellchecker throughout flux-accounting. Like flux-core, I've removed IPv4 and IPv6 from the dictionary. Finally, I've added "dne", "DNE", and "fairshare" as acceptable words.

cmoussa1 commented 3 weeks ago

Thanks @garlick and @jameshcorbett for the review! I've gone ahead and split out the changes into separate commits to 1) fix the spelling mistakes, 2) fix the spellchecker, and 3) add the spellchecker as a builder. I can set MWP here shortly