Skip to content

ci: add website-scoped typo checks#566

Merged
PLeVasseur merged 6 commits intorustfoundation:mainfrom
PLeVasseur:chore/add-typos-ci
Feb 17, 2026
Merged

ci: add website-scoped typo checks#566
PLeVasseur merged 6 commits intorustfoundation:mainfrom
PLeVasseur:chore/add-typos-ci

Conversation

@PLeVasseur
Copy link
Collaborator

@PLeVasseur PLeVasseur commented Feb 16, 2026

Summary

  • add a dedicated check_typos workflow using crate-ci/typos@v1.40.0
  • scope typo checks to arewesafetycriticalyet.org/ for an incremental rollout
  • add _typos.toml, fix one detected website typo, and document local preflight in CONTRIBUTING.md

Validation

  • ran typos arewesafetycriticalyet.org on my fork
  • verified the GitHub Actions check_typos job fails on an intentional typo and passes after revert

@netlify
Copy link

netlify bot commented Feb 16, 2026

Deploy Preview for safety-critical-rust-consortium ready!

Name Link
🔨 Latest commit f2314e6
🔍 Latest deploy log https://app.netlify.com/projects/safety-critical-rust-consortium/deploys/699342cd35a44400085b5fe4
😎 Deploy Preview https://deploy-preview-566--safety-critical-rust-consortium.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Collaborator

@plaindocs plaindocs left a comment

Choose a reason for hiding this comment

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

:shipit: (I don't have sufficient permissions for that to be binding though)

@PLeVasseur
Copy link
Collaborator Author

:shipit: (I don't have sufficient permissions for that to be binding though)

Add triage privs for you @plaindocs 🖖

Thanks for the review :)

@PLeVasseur PLeVasseur merged commit 5ea454b into rustfoundation:main Feb 17, 2026
5 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

Comments