libre-man / unix-opts

Unix-style command line options parser
MIT License
107 stars 13 forks source link

Changed alignment of options in descriptions. #21

Closed cage2 closed 4 years ago

cage2 commented 4 years ago

Hi!

I found the alignment of option's description a bit messed up honestly.

I tried to fix that, i tested with long meta argument or with descriptions that included newlines and seems OK to me.

The only thing i can not understand is why i had to change the directive "~&" to "~%" to make things works.

I am open to critics and review for this patch (i mean, more than usual!).

Bye! C.

libre-man commented 4 years ago

Hi @cage2,

Thanks for the pull request! I'll look at it in the next few days!

cage2 commented 4 years ago

On Sun, Apr 19, 2020 at 09:50:16AM -0700, Thomas Schaper wrote:

Hi @cage2,

Hi!

Thanks for the pull request! I'll look at it in the next few days!

Thank you very much for your work and your quick reply! :)

Bye! C.

cage2 commented 4 years ago

On Sat, May 23, 2020 at 03:08:29AM -0700, Thomas Schaper wrote:

@libre-man commented on this pull request.

Hi!

Thanks for the changes, I've merged it in e14c9b5 after doing some minor changes and adding some tests!

Thanks to you for your review and clearing my doubts about the format directive! :)

Bye! C.