require-ascii: Allow extended ASCII characters

jumanjihouse / pre-commit-hooks

git pre-commit hooks that work with http://pre-commit.com/

MIT License

114 stars 52 forks source link

require-ascii: Allow extended ASCII characters #63

Closed kenorb closed 4 years ago

kenorb commented 4 years ago

Check file encoding.........................................................Failed
- hook id: require-ascii
- exit code: 1

README.md: line 270 column 11 character "©" (decimal 169)

I'd like to exclude the copyright character from README ("©"), is that possible, without disabling the whole hook?

kenorb commented 4 years ago

Also, isn't © part of ASCII, but extended?

See: https://theasciicode.com.ar/extended-ascii-code/copyright-symbol-ascii-code-184.html

jumanjiman commented 4 years ago

Also, isn't © part of ASCII, but extended?

great point!

I've opened https://github.com/jumanjihouse/pre-commit-hooks/issues/62 to allow the whole set of extended ascii characters.

jumanjiman commented 4 years ago

@kenorb

Rev 2.1.4 allows the whole extended ascii set