Skip to content

Conversation

@stewartjarod
Copy link
Contributor

This is a quick fix for the letter spacing for the author list.
It works a little bit better...

This is a quick fix for the letter spacing for the author list.
@hemanth
Copy link
Collaborator

hemanth commented Jul 6, 2014

@stewartjarod Could provide an example where it's not spaced properly with just %-30s ?

@nshtg
Copy link

nshtg commented Jul 6, 2014

I can (funny coincidence):

   $ git summary

 project  : git-plus
 repo age : 3 months
 active   : 51 days
 commits  : 294
 files    : 55
 authors  : 
   141  akonwi                  48.0%
   106  Maximilian Schüßler   36.1%
    26  Akonwi Ngoh             8.8%
     5  Snake231088             1.7%
     4  Ian Huston              1.4%
     2  Joaquín Miguez         0.7%
     2  Paolo G. Giarrusso      0.7%
     2  jigsaw                  0.7%
     2  ravi                    0.7%
     1  Giacomo Trezzi          0.3%
     1  Ilan Biala              0.3%
     1  Michael McCaskill       0.3%
     1  Tam Nguyen              0.3%

https://github.com/akonwi/git-plus

With his fix it alignes nicely but they are tabs and tabs are evil right?

@jbnicolai
Copy link

Couldn't this just be piped through column -t?

@hemanth
Copy link
Collaborator

hemanth commented Jul 7, 2014

@jbnicolai That's a good option. //cc @stewartjarod

hemanth added a commit that referenced this pull request Jul 14, 2014
Letter spacing in author list.
@hemanth hemanth merged commit 04d9dec into tj:master Jul 14, 2014
@jbnicolai
Copy link

@hemanth why the merge? Would you like me to prepare a PR fixing the alignment using column -t?

@hemanth
Copy link
Collaborator

hemanth commented Jul 14, 2014

@jbnicolai That would be great if you could sent a PR. (I checked for few scenarios and found that it was doing the job, so merged)

@jbnicolai
Copy link

👍

tardypad pushed a commit to tardypad/git-extras that referenced this pull request Jul 13, 2016
Letter spacing in author list.
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.

4 participants