alias: mark some common aliases non-legacy

Summary:
People with muscle memories would be a bit surprised that certain commands no
longer work with `sl`. By moving them back from `legacyaliases` to `aliases` the
migration from `hg` to `sl` could be more smooth.

Reviewed By: zzl0

Differential Revision: D49879280

fbshipit-source-id: 20c747d7c44c1c53a89fc425187eb2280134a62d

eden/scm/sapling/commands/__init__.py

@@ -1211,7 +1211,7 @@ def _update_state(repo, state, rev, good, bad, skip):
 
 
 @command(
-    "bookmark|bo|book",
+    "bookmark|bo|book|bookmarks",
     [
         ("f", "force", False, _("force")),
         ("r", "rev", "", _("revision for bookmark action"), _("REV")),
@@ -1222,7 +1222,7 @@ def _update_state(repo, state, rev, good, bad, skip):
     ]
     + formatteropts,
     _("[OPTION]... [NAME]..."),
-    legacyaliases=["bookmarks", "boo", "bookm", "bookma", "bookmar"],
+    legacyaliases=["boo", "bookm", "bookma", "bookmar"],
 )
 def bookmark(ui, repo, *names, **opts):
     """create a new bookmark or list existing bookmarks
@@ -6298,7 +6298,7 @@ def _ensurebaserev(ui, repo, fname):
 
 
 @command(
-    "goto|go",
+    "goto|go|up|update|co|checkout",
     [
         ("C", "clean", None, _("discard uncommitted changes (no backup)")),
         ("c", "check", None, _("require clean working copy")),
@@ -6310,13 +6310,9 @@ def _ensurebaserev(ui, repo, fname):
     ]
     + mergetoolopts,
     legacyaliases=[
-        "update",
-        "up",
         "upd",
         "upda",
         "updat",
-        "checkout",
-        "co",
         "che",
         "chec",
         "check",

eden/scm/sapling/ext/absorb/__init__.py

@@ -963,7 +963,7 @@ def absorb(ui, repo, stack=None, targetctx=None, pats=None, opts=None):
 
 
 @command(
-    "absorb|ab",
+    "absorb|ab|sf",
     [
         (
             "a",
@@ -998,7 +998,7 @@ def absorb(ui, repo, stack=None, targetctx=None, pats=None, opts=None):
     + commands.templateopts
     + commands.walkopts,
     _("@prog@ absorb [OPTION] [FILE]..."),
-    legacyaliases=["abs", "abso", "absor", "sf"],
+    legacyaliases=["abs", "abso", "absor"],
 )
 def absorbcmd(ui, repo, *pats, **opts):
     """intelligently integrate pending changes into current stack