Skip to content

feat: enable crate typos on ci - #357

Open
Maleware wants to merge 3 commits into
mainfrom
feat/enable-crate-typos-on-ci
Open

Maleware wants to merge 3 commits into
mainfrom
feat/enable-crate-typos-on-ci

Conversation

@Maleware

@Maleware Maleware commented Oct 7, 2026

Copy link
Copy Markdown
Member

Adds typo crate and typos.toml according to https://github.com/stackabletech/retro/issues/36

@github-actions

github-actions Bot commented Oct 7, 2026 •

Copy link
Copy Markdown

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 51.08% 2498 / 4890
🔵 Statements 57.16% 3714 / 6497
🔵 Functions 65.83% 1031 / 1566
🔵 Branches 42.96% 1298 / 3021
File CoverageNo changed files found.
Generated in workflow #1057 for commit f6ffbe7 by the Vitest Coverage Report Action

@Maleware Maleware self-assigned this Oct 7, 2026
@Bohreromir
Bohreromir self-requested a review October 7, 2026 09:51

@Bohreromir Bohreromir left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Very good idea, looking at who already introduced a typo 👀

Comment thread typos.toml
# hook in .pre-commit-config.yaml.

[files]
# Bare `typos` skips hidden dirs by default, but prek passes explicit paths and

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

no prek in this repo

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah! Then this needs to look a little bit different I guess. I'll redo.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Development: In Review

Development

Successfully merging this pull request may close these issues.

2 participants