Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

26 new fixes for substitutions.json #24

Closed
wants to merge 26 commits into from

Conversation

CrackedP0t
Copy link
Contributor

I just finished reading Worm for the first time, and I loved it! Thanks so much for making this scraper. While I was reading, I bookmarked all the typos I found, and added them to substitutions.json.

I tried to stick as closely as possible to Wildbow's original writing, and make the fixes as unobtrusive as possible, but I'm absolutely open to other opinions on the changes.

@domenic
Copy link
Owner

domenic commented Jan 1, 2021

Hi, thanks so much; this is great!

I don't think we should change the comma style choice (Oxford vs. not), as IMO that's an authorial style choice and depends on the situation. But otherwise these are great!

I'll push a few fixups, including undoing the Oxford comma changes, and then do a merge and release.

{
"before": "real ly",
"after": "really",
"comment": "There's an empty em element in the middle of this word that gets converted to a space"
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice find with this one. I'll do a larger change to stop adding spaces in place of completely-empty elements; that fixes a lot of extra spaces (although most of those spaces are invisible).

@domenic
Copy link
Owner

domenic commented Jan 1, 2021

Alright, I merged this all as v4.11.0...v4.12.0 and released v4.12.0! Thanks so much!

@domenic domenic closed this Jan 1, 2021
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.

2 participants