Add spelling check script for the CI#10553
Conversation
In order to add the lookahead features and make it easier to exclude specific paths.
|
Oh I really like this approach 👏🏽. IMHO separating the configuration from the script itself is 🔝 I haven't tried it yet, but let me know when its ready to be reviewed and I'll give it an spin. |
|
@andreslucena I think this is now otherwise ready but I will still wait for #10557 to be merged before marking this ready for review. To fix the CI in this PR as it is broken right now. But meanwhile it also gives you a chance to see this in action at the current state of the PR as there are some annotations added at the "Files" tab of this PR right now. EDIT: It has been now merged and the CI is therefore passing in this PR. |
- Extract the correct invalid lines for each check - Add testing also for matching the correct locale files and excluding other locales than `en.yml`
|
@andreslucena I've applied the suggestions from shellcheck and disabled the SC2002 rule (useless cat) as it's incorrect. The cat there is not useless (see e96c46f). |
🎩 What? Why?
As a follow up for #10520, I'd like to experiment with the bash approach for the spelling check rules.
See the comments at #10520 for further information.
📌 Related Issues
Testing
See: https://github.com/ahukkanen/decidim-spellcheck-testing/pull/1/files
📷 Screenshots