Skip to content

Conversation

yarikoptic
Copy link

More about codespell: https://github.com/codespell-project/codespell .

I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.

CI workflow has 'permissions' set only to 'read' so also should be safe.

…nteractively

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "codespell -w -i 3 -C 4",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
@yarikoptic yarikoptic requested a review from a team as a code owner September 9, 2025 18:43
@Copilot Copilot AI review requested due to automatic review settings September 9, 2025 18:43
Copy link
Contributor

@Copilot 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

This PR introduces codespell support to the project to automatically detect spelling errors in the codebase. The changes add a configuration file for codespell settings and a GitHub Actions workflow to run spell checks on pushes and pull requests.

  • Adds codespell configuration with appropriate exclusions for git files and checksums
  • Creates a GitHub Actions workflow to run codespell checks with read-only permissions
  • Fixes existing typos found in comment headers (correcting "Ths" to "The")

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.codespellrc Configuration file defining codespell behavior and exclusions
.github/workflows/codespell.yml GitHub Actions workflow to run spell checks on CI
internal/githubv4mock/query.go Fixes typo in comment header
internal/githubv4mock/local_round_tripper.go Fixes typo in comment header

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@osamamaamon1-jpg
Copy link

osamamaamon1-jpg commented Sep 18, 2025 via email

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