Skip to content

Commit

Permalink
clients/js: fix aptos upgrade and migrate commands
Browse files Browse the repository at this point in the history
For some reason the bound variables in the lambda were underscored,
unclear why. In turn, `yargs` referred to the package, resulting in
runtime errors. This is the sort of thing that an actual type system
would help with.
  • Loading branch information
kcsongor authored and evan-gray committed Aug 28, 2023
1 parent 79e255f commit 73fe659
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions clients/js/src/cmds/aptos.ts
Original file line number Diff line number Diff line change
Expand Up @@ -324,7 +324,7 @@ export const builder = (y: typeof yargs) =>
.command(
"upgrade <package-dir>",
"Perform upgrade after VAA has been submitted",
(_yargs) =>
(yargs) =>
yargs
.positional("package-dir", {
type: "string",
Expand Down Expand Up @@ -365,7 +365,7 @@ export const builder = (y: typeof yargs) =>
.command(
"migrate",
"Perform migration after contract upgrade",
(_yargs) =>
(yargs) =>
yargs
// TODO(csongor): once the sdk has the addresses, just look that up
// based on the module
Expand Down

0 comments on commit 73fe659

Please sign in to comment.