davecheney / httpstat

It's like curl -v, with colours.
MIT License
6.97k stars 382 forks source link

Add -I flag #3

Closed n10v closed 7 years ago

n10v commented 7 years ago

Close https://github.com/davecheney/httpstat/issues/1

n10v commented 7 years ago

@moorereason yes, thanks. I will add it

davecheney commented 7 years ago

The standard flag package only supports single hypen arguments even thought the names of the flags can be longer than a single charater.

I'd prefer this one was called -I. If you want to look at a replacement flag parser that supports BSD and GNU long arguments, that would be fine, in another PR.

On Sat, Sep 24, 2016 at 7:59 AM, Albert notifications@github.com wrote:

@bogem commented on this pull request.

In main.go https://github.com/davecheney/httpstat/pull/3:

)

+func init() {

  • flag.BoolVar(&onlyHeader, "head", false, "show only header of response")

It's the same flag: -I, --head Show document info only But I can name it -I

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/davecheney/httpstat/pull/3, or mute the thread https://github.com/notifications/unsubscribe-auth/AAAcAyQRMh1kOmMUMeFurntoI8lBmU7Pks5qtEu7gaJpZM4KFCag .

n10v commented 7 years ago

@davecheney ok, I will use single character flag. Yes, it would be great to use GNU arguments. I'll open a discussion about that in another PR later this day

n10v commented 7 years ago

🙏 All conflicts are solved