Flatten package commands #240

tnowacki · 2022-07-05T17:45:39Z

Made all package commands base-line
Should make usage of the move cli a bit more natural

Motivation

Splitting into separate command structure like this will make the reusing of the commands a lot easier
And with other CLI tools re-using the commands, they can easy add extra/custom args with #[clap(flatten)] and a separate struct

Test Plan

updated tests

- Made all package commands base-line

awelc

LGTM

* Flatten package commands - Made all package commands base-line

Flatten package commands

02f95af

- Made all package commands base-line

tnowacki requested review from davidiw, awelc, sblackshear, damirka and vgao1996 July 5, 2022 17:45

tnowacki added 4 commits July 5, 2022 11:05

fixup! Flatten package commands

eb902c1

fixup! fixup! Flatten package commands

589af4d

fixup! fixup! fixup! Flatten package commands

ed8c780

fixup! fixup! fixup! fixup! Flatten package commands

6bdc216

awelc approved these changes Jul 5, 2022

View reviewed changes

tnowacki merged commit c37321b into move-language:main Jul 5, 2022

tnowacki deleted the flatten branch July 5, 2022 19:53

awelc mentioned this pull request Jul 6, 2022

[Move CLI] Updated move CLI to work with flattened pkg core Move command MystenLabs/sui#3019

Merged

villesundell pushed a commit to villesundell/move that referenced this pull request Jul 7, 2022

Flatten package commands (move-language#240)

89b2f8f

* Flatten package commands - Made all package commands base-line

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Flatten package commands #240

Flatten package commands #240

tnowacki commented Jul 5, 2022 •

edited

Loading

awelc left a comment

Flatten package commands #240

Flatten package commands #240

Conversation

tnowacki commented Jul 5, 2022 • edited Loading

Motivation

Test Plan

awelc left a comment

Choose a reason for hiding this comment

tnowacki commented Jul 5, 2022 •

edited

Loading