Skip to content

Keep spellchecking local and run lint in CI - #63

Merged
drbruced12 merged 7 commits into
SystemsApproach:mainfrom
Walentalien:fix/improve-ci1
Sep 28, 2026
Merged

drbruced12 merged 7 commits into
SystemsApproach:mainfrom
Walentalien:fix/improve-ci1

Conversation

@Walentalien

Copy link
Copy Markdown
Contributor

No description provided.

Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
@Walentalien
Walentalien marked this pull request as draft September 26, 2026 09:35
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
@Walentalien

Copy link
Copy Markdown
Contributor Author

@drbruced12
Ubuntu uses 2020 version of dictionary and spellcheck fails on words that are added in newer version (that Arch pins for example)

The solution is either to add those words to dict.txt OR use arch container in ci (can also install latest dictionary with a script but it's not best solution)
I'd just use Arch container.

Do you care about spellchecker in ci at all ?

@llpeterson

Copy link
Copy Markdown
Contributor

I'm not anxious to complicate our CI process. It's great that contributors can spell check their contributions (and I'm happy to have it working again for myself), but automatically including it in the pipeline isn't a requirement.

Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
Signed-off-by: Walentalien <voliinyk@student.agh.edu.pl>
@Walentalien
Walentalien marked this pull request as ready for review September 28, 2026 09:17
@Walentalien

Copy link
Copy Markdown
Contributor Author

I'm not anxious to complicate our CI process. It's great that contributors can spell check their contributions (and I'm happy to have it working again for myself), but automatically including it in the pipeline isn't a requirement.

That makes sense. CI was already running spelling check through make test, but it wasn't failing the job. I've changed CI to run lint and build (running spellcheck is for local usage now)

@Walentalien Walentalien changed the title Make CI fail on spellchecker failure Keep spellchecking local and run lint in CI Sep 28, 2026
@drbruced12

Copy link
Copy Markdown
Contributor

Thanks for explaining the dictionary issue - I was a bit mystified by the failure of "dahl". I think we are all on the same page now so going ahead with merge.

@drbruced12
drbruced12 merged commit 11418f0 into SystemsApproach:main Sep 28, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants