Skip to content
Snippets Groups Projects
Commit 0ca98ed8 authored by Pulkit Goyal's avatar Pulkit Goyal
Browse files

upgrade: remove unnecessary `is None` check

`upgrade_engine.upgrade()` always return the `backuppath` value and there are
not early returns. Hence I don't see how `backuppath` can be None.

Adding extra unncessary safe checks hides unknown bugs. Hence removing it.

Differential Revision: https://phab.mercurial-scm.org/D9675
parent 1ca7865c
No related branches found
No related tags found
No related merge requests found
...@@ -205,7 +205,7 @@ ...@@ -205,7 +205,7 @@
backuppath = upgrade_engine.upgrade( backuppath = upgrade_engine.upgrade(
ui, repo, dstrepo, upgrade_op ui, repo, dstrepo, upgrade_op
) )
if not (backup or backuppath is None): if not backup:
ui.status( ui.status(
_(b'removing old repository content %s\n') % backuppath _(b'removing old repository content %s\n') % backuppath
) )
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment