carrot / ship

⛔️ currently unmaintained ⛔️
Other
151 stars 14 forks source link

args.js #17

Closed hhsnopek closed 10 years ago

hhsnopek commented 10 years ago

I notice @jenius you starred this repo, and I ended up looking into it. It seems like it could be a better alternative to our current arg_parser, I could add this in as I'm currently re-organizing ship; thoughts?

jescalan commented 10 years ago

Arg parser is very specific to ship, it's not really a general purpose thing. If you can clean it up while retaining all functionality though, would love to see an integration

notslang commented 10 years ago

I think you're a little confused about what args.js does... it's for function arguments, not parsing command-line args. But, the idea of moving arg-parsing into a library is a great one, so I opened an issue here: https://github.com/carrot/ship/issues/24

hhsnopek commented 10 years ago

It can do both from what I understood

notslang commented 10 years ago

can it?

hhsnopek commented 10 years ago

This is the arg.js I was talking about: autographer/args.js, my understanding was that it can check against arguments passed, verify and spit out info... Yet I did remove it from my re-org. I do have to say the way @jenius set up the arg_parser is beautiful, might of been why I removed arg.js