FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Add remove-em-dash hook by JakubJablonski2-TomTom · Pull Request #1263 · pre-commit/pre-commit-hooks · GitHub

Add remove-em-dash hook - #1263

Closed
JakubJablonski2-TomTom wants to merge 1 commit into
pre-commit:mainfrom
JakubJablonski2-TomTom:add-remove-em-dash-hook
Closed

Add remove-em-dash hook#1263
JakubJablonski2-TomTom wants to merge 1 commit into
pre-commit:mainfrom
JakubJablonski2-TomTom:add-remove-em-dash-hook

Conversation

Copy link
Copy Markdown

New fixer hook that replaces UTF-8 em-dashes (U+2014) with a plain hyphen (-), modeled on the trailing-whitespace hook.

  • pre_commit_hooks/remove_em_dash.py: the fixer (binary-safe, UTF-8 only)
  • tests/remove_em_dash_test.py: full coverage of fix and no-op cases
  • registered in setup.cfg, .pre-commit-hooks.yaml, and README.md

New fixer hook that replaces UTF-8 em-dashes (U+2014) with a plain
hyphen (-), modeled on the trailing-whitespace hook.

- pre_commit_hooks/remove_em_dash.py: the fixer (binary-safe, UTF-8 only)
- tests/remove_em_dash_test.py: full coverage of fix and no-op cases
- registered in setup.cfg, .pre-commit-hooks.yaml, and README.md

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Copy link
Copy Markdown
Member
  • we don't accept features without prior discussion
  • we don't accept ai slop
  • this doesn't seem very useful anyway

asottile closed this Jun 24, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL