Skip to content

Add typos to Invidious#5356

Open
syeopite wants to merge 7 commits intoiv-org:masterfrom
syeopite:add-typos
Open

Add typos to Invidious#5356
syeopite wants to merge 7 commits intoiv-org:masterfrom
syeopite:add-typos

Conversation

@syeopite
Copy link
Member

Serious title: Add job to check spelling via typos tool as apart of CI


I recently learned that ameba has integrations with the typos tool to detect spelling mistakes within the source code, though it does need to be installed first but I'm not exactly sure how to do that so it'll be done in a separate job for now.

The next release of the Ameba Github Actions program (which should include the typos tool) should add support to create annotations without needing a token so we can just merge both the Ameba and spell check jobs into one once we switch to the action program.

@unixfox
Copy link
Member

unixfox commented Sep 8, 2025

(Now copilot can scan for typos ^^) iv-org/invidious-companion#107 (comment)

Copy link
Member

@Fijxu Fijxu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is ok, I personally prefer to leave the work to this job instead of Copilot so we can easily move Invidious from one platform to another while preserving the typos correction.

@syeopite
Copy link
Member Author

image

Maybe CHANGELOG.md should also be excluded?

@Fijxu
Copy link
Member

Fijxu commented Jan 30, 2026

Maybe CHANGELOG.md should also be excluded?

Yes

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.

4 participants