Skip to content

Add typos to Invidious #5356

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

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Add typos to Invidious #5356

wants to merge 3 commits into from

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.

@@ -165,3 +165,12 @@ jobs:

- name: Run Ameba linter
run: bin/ameba

lint_spell:
name: Spell Check with Typos
Copy link
Member Author

Choose a reason for hiding this comment

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

Suggested change
name: Spell Check with Typos
name: Check spelling

* UI: Add search button to search bar ([#4706], thanks @thansk)
* UI: Add search button to search bar ([#4706], thanks @thanks)
Copy link
Member

Choose a reason for hiding this comment

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

You might want to add an exception for github handles

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.

2 participants