-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Proposal: Add Markdown related jobs to flowzone #299
Comments
Good idea, then we don't have to deal with upstream misspellings. Wouldn't want custom dictionary words to be a blocker for PRs across all the repos though so need an idea around that. cspell has an add word feature, not sure if that's feasible to have as part of the CI to add words on the fly during tests. @vipulgupta2048 |
Who do we reach out to have a discussion on this first? I was hoping to get more comments in first before rushing to execute it. I know it will be an improvement from what we currently have but let's get this done right. Spellchecker Questions
Linkchecker questions
|
Adding this here as examples of remote dictionaries.
|
Should we trial cspell just in docs for a month or so before flowzone roll out? |
SGTM, I might even bring this up in arch call? The question of having Markdown related checks even worth or not is probably worth discussing |
I think that's what we are doing. |
Would it be a good idea to add markdown related flows to flowzone?
The docs runs a spellchecker using a tool called Cspell and a link checker called lychee.
IMHO these checks could be really helpful for repos like handbook, balena-site, masterclasses or any product repo holding markdown docs.
Usecases:
The text was updated successfully, but these errors were encountered: