Page MenuHomePhabricator

basestore: use tryunlink from mercurial.util to try and remove files
ClosedPublic

Authored by singhsrb on Dec 1 2017, 2:34 PM.

Details

Summary

This commit just refactors to use a method meant for the same
purpose. Thanks to @quark for discovering this method!

Test Plan

Ran all the tests.

Diff Detail

Repository
rFBHGX Facebook Mercurial Extensions
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

singhsrb created this revision.Dec 1 2017, 2:34 PM
Herald added a reviewer: Restricted Project. · View Herald TranscriptDec 1 2017, 2:34 PM
quark accepted this revision.Dec 1 2017, 2:41 PM
This revision is now accepted and ready to land.Dec 1 2017, 2:41 PM
This revision was automatically updated to reflect the committed changes.