Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added Basic CLI #95

Merged
merged 5 commits into from
Aug 25, 2017
Merged

Added Basic CLI #95

merged 5 commits into from
Aug 25, 2017

Conversation

ccpandhare
Copy link
Collaborator

@ccpandhare ccpandhare commented Aug 24, 2017

I have added basic CLI.
The main file is ./index.js

Usage

$ ./index.js OPTIONS

Options

    -V, --version           output the version number
    -i, --image [PATH/URL]  Input image URL
    -s, --step [step-name]  Name of the step to be added.
    -o, --output [PATH]     Directory where output will be stored.
    -h, --help              output usage information

-i and -s are required flags. not specifying either will throw an error and call process.exit(1)

@jywarren
Copy link
Member

oooooooooooooo 🎈

@ccpandhare
Copy link
Collaborator Author

ccpandhare commented Aug 24, 2017

I will make keep on making changes on this PR.
next, I shall add support for the user specified options.

@ccpandhare
Copy link
Collaborator Author

Then I will add the rest as FTO issues as you suggested earlier.

@jywarren
Copy link
Member

jywarren commented Aug 24, 2017

@nadya check this out! commandline interface!

@jywarren
Copy link
Member

Yes and let's add a CLI section to the README to show just a simple example of the usage, like:

$ ./index.js -i file.jpg -s "invert"

What is the filneame of the output?

@ccpandhare
Copy link
Collaborator Author

I have added the outputs feature. Now modules' text outputs are shown in this manner:

➜  ./index.js -i examples/images/IS-QR.png -s decode-qr
---
Files will be exported to "./output/"
[decode-qr]: qrval = http://github.com/publiclab/image-sequencer
---

@ccpandhare
Copy link
Collaborator Author

Since CLI is the least developed part of image sequencer, I see immense opprtunity of creating FTOs here. As you had suggested earlier @jywarren, I think It would be a good idea to shift development to other issues and add some of the semaining CLI features as FTOs. Sounds cool?

@ccpandhare
Copy link
Collaborator Author

If yes, then I think we should merge this in and then I can start writing FTOs for CLI!

@jywarren
Copy link
Member

jywarren commented Aug 25, 2017 via email

@ccpandhare ccpandhare merged commit d466afb into publiclab:master Aug 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants