Multiplayer
Multiplayer copied to clipboard
Remove String Method Names from Harmony Patches
Labels: Enhancement, Good first issue
Some method names are still written as string literals instead of using nameof(Thing.Tick). This makes it harder to quickly locate patched methods.
It would be great to update all patches to consistently use the nameof convention.
Example:
https://github.com/rwmt/Multiplayer/blob/f0221d0488282629ebbdef7d58c9d4a93e393b85/Source/Client/MultiplayerStatic.cs#L376-L384
Hi, I’d like to work on this issue. Can I be assigned?
You don't need permission, go for it. The task is pretty simple, find Method names in strings and replace them with their nameof counterpart.