Skip to content

Conversation

uglide
Copy link
Contributor

@uglide uglide commented Oct 7, 2025

Nowadays, AI reviewing tools like CoPilot catch spellcheck issues better than pyspelling without a burden of maintaining wordlist.txt

@uglide uglide requested a review from Copilot October 7, 2025 08:21
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 removes the spellcheck workflow and related configuration files from the repository. The change simplifies the CI/CD pipeline by eliminating the pyspelling-based spellcheck system in favor of relying on AI-powered tools like GitHub Copilot for spell checking.

  • Removes the GitHub Actions spellcheck workflow
  • Deletes the extensive wordlist dictionary with 358+ entries
  • Removes the pyspelling configuration file

Reviewed Changes

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

File Description
.github/workflows/spellcheck.yml Removes the entire spellcheck GitHub Actions workflow
.github/wordlist.txt Removes the comprehensive dictionary of allowed words and technical terms
.github/spellcheck-settings.yml Removes the pyspelling configuration for markdown processing

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

@uglide uglide requested review from atakavci and ggivo October 7, 2025 08:22
Copy link

github-actions bot commented Oct 7, 2025

Test Results

   283 files  ±0    283 suites  ±0   11m 24s ⏱️ -13s
10 075 tests ±0  9 019 ✅  - 1 011  1 056 💤 +1 011  0 ❌ ±0 
 2 713 runs  ±0  2 713 ✅ ±    0      0 💤 ±    0  0 ❌ ±0 

Results for commit 8d042b3. ± Comparison against base commit 6a4a94b.

This pull request skips 1011 tests.
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetdel
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetdelBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHgetexBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHsetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[1] ‑ testHsetexBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetdel
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetdelBinary
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetex
redis.clients.jedis.commands.commandobjects.CommandObjectsHashCommandsTest[2] ‑ testHgetexBinary
…

♻️ This comment has been updated with latest results.

Nowadays, AI reviewing tools like CoPilot catch spellcheck issues better than pyspelling without a burden of maintaining wordlist.txt
@uglide uglide force-pushed the im/remove-spellchecker branch from 4e64d40 to 8d042b3 Compare October 8, 2025 13:18
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