Skip to content

Conversation

@riderx
Copy link

@riderx riderx commented Nov 11, 2025

Introduce a GitHub Actions workflow to automatically check for typos in pull requests. This enhances code quality by ensuring that typographical errors are caught early in the review process.

@vercel
Copy link

vercel bot commented Nov 11, 2025

@riderx is attempting to deploy a commit to the Ionic Team on Vercel.

A member of the Team first needs to authorize it.

Copilot AI review requested due to automatic review settings February 10, 2026 02:12
Copy link

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

Adds a GitHub Actions workflow to run automated typo detection on pull requests to catch spelling issues during review.

Changes:

  • Introduce a new typos workflow triggered on PR open/sync/reopen.
  • Run crate-ci/typos after checking out the repository.

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


jobs:
typos:
name: Check for typos
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

Consider adding explicit minimal permissions (e.g., contents: read) for this workflow/job. The repo’s other workflow sets permissions explicitly, and doing so avoids relying on repository defaults and reduces token scope.

Suggested change
name: Check for typos
name: Check for typos
permissions:
contents: read

Copilot uses AI. Check for mistakes.
Comment on lines +8 to +11
typos:
name: Check for typos
runs-on: ubuntu-latest
steps:
Copy link

Copilot AI Feb 10, 2026

Choose a reason for hiding this comment

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

Add timeout-minutes to the job for consistency with other workflows in this repo and to prevent a stuck runner from consuming CI capacity indefinitely.

Copilot uses AI. Check for mistakes.
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.

1 participant