Skip to content

Conversation

@JR-1991
Copy link
Member

@JR-1991 JR-1991 commented Oct 20, 2023

  • Addition of a command line interface to use DVUploader besides scripts
  • Workflow to generate binaries that bundle the CLI to an executable

Example usage

Using arguments

dvuploader my_file.txt my_other_file.txt \
           --pid doi:10.70122/XXX/XXXXX \
           --api-token XXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX \
           --dataverse-url https://demo.dataverse.org/ \

Using a config file

# config.yml
persistent_id: doi:10.70122/XXX/XXXXX
dataverse_url: https://demo.dataverse.org/
api_token: XXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXXXX
files:
    - filepath: ./small.txt
    - filepath: ./medium.txt
      directoryLabel: some/dir
    - filepath: ./big.txt
      directoryLabel: some/dir

The config file can then be used as follows:

dvuploader --config-path config.yml

@JR-1991
Copy link
Member Author

JR-1991 commented Oct 20, 2023

Note - Checks fail because workers cannot push the resulting binaries to the repo. This can be mitigated by enabling actions to push to repositories within the gdcc organization settings.

@pdurbin is this possible or a potential security issue?

@JR-1991 JR-1991 added the enhancement New feature or request label Oct 20, 2023
@JR-1991 JR-1991 self-assigned this Oct 20, 2023
@pdurbin
Copy link
Member

pdurbin commented Oct 20, 2023

@JR-1991 for some reason you only had "write" on this repo. I just upgraded you to "admin". Can you please try again?

@pdurbin
Copy link
Member

pdurbin commented Oct 27, 2023

I just tried the quickstart for the first time. Worked great.

Then, I had the thought... there should be a CLI for this.

Then I rediscovered this PR that @JR-1991 is already working on. Great!

@JR-1991 JR-1991 marked this pull request as ready for review November 8, 2023 18:09
@JR-1991 JR-1991 merged commit 91cbf33 into main Nov 8, 2023
@JR-1991 JR-1991 deleted the feature/cli branch November 26, 2023 13:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants