Skip to content

chore: Add typos to pipeline - #30

Merged
mhambre merged 1 commit into
masterfrom
chore/add-typo-to-pipeline
Aug 27, 2026
Merged

chore: Add typos to pipeline#30
mhambre merged 1 commit into
masterfrom
chore/add-typo-to-pipeline

Conversation

@mhambre

@mhambre mhambre commented Aug 27, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI lite review requested due to automatic review settings August 27, 2026 04:15

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟢 Approval recommended

The changes are straightforward CI/documentation updates with consistent configuration and no apparent functional risk.

Pull request overview

This PR extends the repository’s CI quality gate by adding automated spelling checks via typos, and updates documentation to reflect the new pipeline step plus a small CAS diagram sizing tweak.

Changes:

  • Add a typos install + run step (pinned to 1.49.0) to the reusable quality workflow.
  • Document the new typos check in docs/CI.md.
  • Add a _typos.toml config to exclude .excalidraw files from spelling checks; adjust CAS diagram image width.
File summaries
File Description
docs/CI.md Documents the new typos CI step and links to the tool.
docs/architecture/CAS.md Adjusts the CAS diagram display width in the architecture doc.
.github/workflows/quality.yml Installs and runs typos as part of the quality workflow with restricted token permissions.
_typos.toml Configures typos exclusions for .excalidraw files.
Review details
  • Files reviewed: 4/4 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@mhambre
mhambre merged commit af0557a into master Aug 27, 2026
13 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.

2 participants