Proper/Improper noun replacement proposal

[SabreML]

Details:

• Files changed: 260
• Line insertions: 811
• Line deletions: 811
• Potential labels: Grammar and Formatting, Tweak/Fix

---

I've gone through every file I could find in the code with name = "Uppercase Name" and replaced every instance (where needed) with either a lowercase version, or added an \improper so that it displays correctly in-game.

This regex was used to search, but each replacement was done manually to avoid false positives:

\tname\s*=\s*"\s*[A-Z].*"

 

Now obviously this is a lot of changes to go through at once, so my current plan (assuming there's no objections) is to slowly post all of this in multiple smaller 'themed' PRs rather than in one massive one. That way I can go through everything in the 'Weapons' or 'Machinery' categories for example, and double check that I didn't make a mistake somewhere.
Most importantly, this would also have the benefit of making the PRs actually possible to review, because while each change is very easy to check individually, 811 of them at once is not.

Alternatively I could just split this into 130-file chunks, but atomising it seems like the best way to go.

---

Examples: (Before/After)

-----

-----

-----

-----

-----

-----

---

PRs:

1. https://github.com/ParadiseSS13/Paradise/pull/16897
2. https://github.com/ParadiseSS13/Paradise/pull/16906

Edited October 9, 2021 by SabreML

[AffectedArc07]

3 hours ago, SabreML said:

This regex was used to search, but each replacement was done manually to avoid false positives:

I hope so because mass regex replacements are a gateway to "no thanks"

 

3 hours ago, SabreML said:

Now obviously this is a lot of changes to go through at once, so my current plan (assuming there's no objections) is to slowly post all of this in multiple smaller 'themed' PRs rather than in one massive one.

Please do this

 

 

 

TLDR: Approved, but you have to split this to manageable chunks, or its getting shot.