search

wiremod / advduplicator

Garry's Mod add-on that allows a player to save and load contraptions
http://www.wiremod.com
Apache License 2.0
67 stars 43 forks source link

Lowercased all file operations #43

Closed Nebual closed 11 years ago

Nebual commented 11 years ago

Fixes #39

Anticept commented 11 years ago

Out of curiosity, do we have any formal policy on merging? I noticed sometimes we can have pull requests open for days, even weeks, before someone says "Should we merge it?"

Divran commented 11 years ago

If laziness is a formal policy, then yes

Nebual commented 11 years ago

In this case, I cannot test that this actually resolves the problem, as I do not have a Linux server, though I can say its still fine on Windows. In other cases, I'm usually worried that my solution is not tidy enough, so I leave it open for a few days in case anyone else has a better solution, to avoid "Neb why did you rename all those files the wrong way, now their history is (almost) lost"

Anticept commented 11 years ago

Due to the importance of this pull request though, if it looks OK, lets get it pulled and published!

Anticept commented 11 years ago

Guess it's trial by fire then!