Skip to content

Removed --encoding option#69

Merged
twolfson merged 8 commits intomasterfrom
todd.remove.encoding.option
Nov 22, 2025
Merged

Removed --encoding option#69
twolfson merged 8 commits intomasterfrom
todd.remove.encoding.option

Conversation

@twolfson
Copy link
Copy Markdown
Owner

As discussed in #65, the --encoding CLI option requiring to be known and opted into is confusing.

i.e. We shouldn't really care about all the file contents, we should only care about the reST parts.

In this PR:

@twolfson
Copy link
Copy Markdown
Owner Author

LGTM :shipit:

@twolfson twolfson merged commit 7b43036 into master Nov 22, 2025
5 checks passed
@twolfson twolfson deleted the todd.remove.encoding.option branch November 22, 2025 03:46
tdenewiler added a commit to tdenewiler/statick that referenced this pull request Nov 25, 2025
The upstream package changed their API in
<twolfson/restructuredtext-lint#69> to remove
the `encoding` argument.
tdenewiler added a commit to triboelectric/statick that referenced this pull request Dec 2, 2025
The upstream package changed their API in
twolfson/restructuredtext-lint#69 to remove the
`encoding` argument. The change made it into the 2.0.0 release.
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.

UnicodeDecodeError: 'charmap' codec can't decode byte 0x9d

1 participant