move icon indicating copy to clipboard operation
move copied to clipboard

[Movey] Add command to upload metadata to Movey

Open ea-open-source opened this issue 2 years ago • 2 comments

Motivation

Add move movey-upload command to upload move package to Movey, modify move build command to call Movey API to increase download count.

How to use it

Before you use the new command to upload metadata to Movey, please make sure:

  • your package must be on Github
  • the remote must be named “origin”
  • file $HOME/.move/movey_api_key.toml must exist, and the content should be like(you can get the token on Movey):
[registry]
token = "example-token"

Note: you could use the move login command from this PR to save your Movey's api token using the CLI.

Step 1: Navigate to the folder containing your package. Step 2: Run the following command: move movey-upload

Have you read the Contributing Guidelines on pull requests?

Yes

Test Plan

cargo test --package move-cli --test cli_tests upload_package cargo test --package move-cli --lib get_api_token

ea-open-source avatar Jun 27 '22 09:06 ea-open-source

wrwg, awelc, damirka Would you guys help us with rerunning Actions and review? We have already contacted Sam and Todd.

ea-open-source avatar Jul 07 '22 12:07 ea-open-source

@sblackshear All issues were resolved, please check again.

ea-open-source avatar Jul 15 '22 02:07 ea-open-source

Hi Sam, we have updated all 3 of our PRs(https://github.com/move-language/move/pull/226, https://github.com/move-language/move/pull/227, https://github.com/move-language/move/pull/318). Please give them another round of review.

ea-open-source avatar Aug 02 '22 11:08 ea-open-source

Hi Sam, we have updated all 3 of our PRs(#226, #227, #318). Please give them another round of review.

I reassigned these to myself and will re-review shortly.

awelc avatar Aug 03 '22 23:08 awelc

I have merged #226 and, as a result, this branch now has conflicts. While resolving these, please refactor Movey configuration file access routines so that the functionality to create/read the file (for both the login command and for the upload command are in the same place).

awelc avatar Aug 08 '22 22:08 awelc