Fix various linter issues

wiremod / wire

Garry's Mod add-on that allows users to wire up components in order to make more elaborate automatic and user-controlled contraptions.

http://www.wiremod.com

Apache License 2.0

554 stars 330 forks source link

Fix various linter issues #3108

Closed wrefgtzweve closed 3 months ago

wrefgtzweve commented 3 months ago

Fixes a load of linter errors, doesn't fix all of them but i didn't want to make the PR too big.

thegrb93 commented 3 months ago

You better double check everything or I'm straight up closing it. I hate reviewing these kinds of PRs.

wrefgtzweve commented 3 months ago

Yeah i can understand that, as for the unused variables it should probably be turned off in the linter then because they show up with wiremods settings. I imagine it'd be better if i did singular changes, eg a PR that only changes // to --.

thegrb93 commented 3 months ago

That would be far easier to review, yeah.

wrefgtzweve commented 3 months ago

👍 I'll do a follow up PR for those sometime then, currently im a bit busy but i'll eventually get around it