Skip to content

Fix typos discovered by codespell#139

Merged
cdgriffith merged 1 commit intocdgriffith:developfrom
cclauss:codespell
Mar 12, 2026
Merged

Fix typos discovered by codespell#139
cdgriffith merged 1 commit intocdgriffith:developfrom
cclauss:codespell

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 9, 2026

Fix typos discovered by codespell - https://pypi.org/project/codespell

% codespell --ignore-words-list=addin,caf,tye,wors --skip="*.json" --quiet-level=3

./puremagic/scanners/mpeg_audio_scanner.py:231: posistion ==> position
./puremagic/scanners/mpeg_audio_scanner.py:469: pratical ==> practical
./puremagic/scanners/mpeg_audio_scanner.py:496: preceeded ==> preceded, proceeded

% codespell --ignore-words-list=addin,caf,tye,wors --skip="*.json" --quiet-level=3 --write-changes

Copilot AI review requested due to automatic review settings March 9, 2026 21:46
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

This PR corrects minor spelling mistakes in MPEG audio tag-scanning comments found by codespell, improving readability without changing runtime behavior.

Changes:

  • Fix “posistion” → “position” in a TAG+ parsing comment.
  • Fix “pratical” → “practical” in an APE tag scan-range comment.
  • Fix “preceeded” → “preceded” in an APEv1 parsing comment.

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

@cdgriffith
Copy link
Owner

Hello, please base this off develop and not master, thanks!

@cclauss cclauss changed the base branch from master to develop March 10, 2026 05:16
@cdgriffith cdgriffith merged commit 19c3586 into cdgriffith:develop Mar 12, 2026
8 of 9 checks passed
@cclauss cclauss deleted the codespell branch March 12, 2026 06:49
@cdgriffith cdgriffith mentioned this pull request Mar 13, 2026
cdgriffith added a commit that referenced this pull request Mar 13, 2026
- Adding #6 `from_extension()` and `magic_extension()` to look up MIME types by file extension
- Adding #119 CFBF/OLE2 scanner to distinguish Word, Excel, PowerPoint, Outlook MSG, Visio, and Publisher files (thanks to Simeon Stoykov and Andy - NebularNerd)
- Adding #139 codespell check (thanks to Christian Clauss)
- Adding `-e`/`--extension` CLI flag for extension-based MIME type lookup
- Fixing #104 WAV MIME type changed from `audio/wave` to `audio/wav` (thanks to Simon Willison and Andy - NebularNerd)
---------

Co-authored-by: Christian Clauss <cclauss@me.com>
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