gitserver: Attempting to delete an already deleted repo dir is safe

Created by: ryanslade

In the previous implementation we would return an error. This caused a cascade of retries which meant we would try to delete the same repo over 20 times.

Instead, just return a nil error which makes this idempotent.

Test plan

Updated unit tests