Page MenuHomePhabricator

shelve: fix a missing variable in the exception handler for delete
ClosedPublic

Authored by mharbison72 on Tue, Nov 19, 5:46 PM.

Details

Summary

Caught by pytype. I haven't paid much attention to the progress of this
extension, but I *think* this was the intent.

Diff Detail

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

Event Timeline

mharbison72 created this revision.Tue, Nov 19, 5:46 PM

This and the next one should probably go on stable. I'm a bit surprised there's no test coverage, at least for the next one.

dlax accepted this revision.Wed, Nov 20, 2:59 AM
dlax added a subscriber: dlax.

I don't see any test for hg shelve --delete NAME that's not a command error case; adding some in D7460.

pulkit accepted this revision.Thu, Nov 21, 7:29 AM
This revision is now accepted and ready to land.Thu, Nov 21, 7:29 AM