Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Very simple command line #14

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Conversation

fadyanwar
Copy link

I've created this command line for personal use since the example main.go didn't take in args for files to upload, so I thought I might be sharing it as well.

@fadyanwar
Copy link
Author

Also implemented one of TODOs to retry on failure in the client library file put.go

@LeslieOA
Copy link

Hey @fadyanwar. If it helps, we have a JS CLI you can use instead:
https://github.com/web3-storage/web3.storage/tree/main/packages/w3.

@fadyanwar
Copy link
Author

Thanks @LeslieOA for the tip, I was already reviewing this repo to see how you guys do it :) So definitely going to check it out. That being said, I had an impression that Go is better when it comes to IO and memory management since it compiles to native code rather than interpreted as in JS. I'm saying this because I already had an issue uploading large files using the JS library as well as the website itself.

@Jorropo
Copy link
Contributor

Jorropo commented Jan 30, 2022

@fadyanwar btw duplicates of #12 :)

I don't know which one of the PRs is better.

@fadyanwar
Copy link
Author

Hi @Jorropo thanks for letting me know. I didn't realize that there was an existing PR with a command line till I wrote this one. Yours appears to have more features i.e. flags, mine was just intended as a simple CLI that takes in just the file name. I did however include a commit for the library itself to retry on failure. Should I seperate it in a new PR?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants