Skip to content

[Improvement] Spell check only PR-changed files and added manual dispatch for full check#693

Merged
LukasWallrich merged 7 commits intomasterfrom
prspellcheck
Mar 12, 2026
Merged

[Improvement] Spell check only PR-changed files and added manual dispatch for full check#693
LukasWallrich merged 7 commits intomasterfrom
prspellcheck

Conversation

@richarddushime
Copy link
Copy Markdown
Contributor

@richarddushime richarddushime commented Mar 8, 2026

Description

This improves the spell check feature and Instead of commenting on every Pull Request , typos from other files

  • PR triggers now only spell-check files changed in the PR (not entire site contents)
  • Added workflow_dispatch with option to check all content manually
  • Updated script to handle both modes with appropriate output messages

Type of Change

  • New feature

Testing

Checklist

  • Self-reviewed my changes
  • Verified links and formatting are correct
  • No new warnings or errors

Notes

@richarddushime richarddushime requested a review from a team as a code owner March 8, 2026 00:42
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 8, 2026

👍 All image files/references (if any) are in webp format, in line with our policy.

@richarddushime richarddushime changed the title [Improvement] Spell check only PR-changed files; add manual dispatch for full check [Improvement] Spell check only PR-changed files and added manual dispatch for full check Mar 8, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 8, 2026

✅ Spell Check Passed

No spelling issues found when checking 2 changed file(s)! 🎉

@LukasWallrich
Copy link
Copy Markdown
Contributor

LukasWallrich commented Mar 8, 2026

Staging Deployment Status

This PR has been successfully deployed to staging as part of an aggregated deployment.

Deployed at: 2026-03-12 20:21:34 UTC
Staging URL: https://staging.forrt.org

The staging site shows the combined state of all compatible open PRs.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the repository’s spell-check automation to support two execution modes (PR-only vs full scan), aiming to reduce CI work on pull requests while still enabling an on-demand full repository check.

Changes:

  • Add PR mode to check_spelling.py that spell-checks only PR-changed files (via CHANGED_FILES) and keep a full-scan mode (via CHECK_ALL).
  • Extend the GitHub Actions workflow to compute changed files for PRs and add workflow_dispatch for manual full checks.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
scripts/spell_check/check_spelling.py Adds mode selection logic (PR-only vs full) and adjusts comment formatting to reflect the mode/files checked.
.github/workflows/spell-check.yaml Adds workflow_dispatch, computes changed files in PR runs, and passes mode-related env vars into the script.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions
Copy link
Copy Markdown
Contributor

✅ Spell Check Passed

No spelling issues found when checking 2 changed file(s)! 🎉

richarddushime and others added 2 commits March 12, 2026 14:39
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@LukasWallrich LukasWallrich merged commit 38f0bdb into master Mar 12, 2026
5 checks passed
@LukasWallrich LukasWallrich deleted the prspellcheck branch March 12, 2026 21:55
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.

3 participants