search

salarkhan / git-pear

swap credentials after each commit
3 stars 0 forks source link

pear breaks on brand new repos #29

Closed salarkhan closed 10 years ago

salarkhan commented 10 years ago

Here's the error: screen shot 2014-07-09 at 12 33 01 am

And here's why: screen shot 2014-07-09 at 12 38 28 am

We need to give a default username. Maybe just a blank space will do?

salarkhan commented 10 years ago

Alternatively, we could forego setting the author for each committer's first commit.

Ex: grep git log --pretty=format:"%an" for an author name. If it doesn't exist, don't attempt to set author.

salarkhan commented 10 years ago

Good news & bad news. Good news, I got it working (don't mind the variable assignment crap, I was trying something else out): screen shot 2014-07-09 at 8 35 01 am

Bad news, the author name is whatever is stored in your default. And that becomes the author name for both pear users. screen shot 2014-07-09 at 8 36 43 am

Not a big deal because the author name doesn't actually do anything. But if that's the case, I think we should just set the author name to Pear by default. Like so: screen shot 2014-07-09 at 8 40 55 am screen shot 2014-07-09 at 8 41 07 am

This of course, is MVP. I think once we get the Github API key we can tackle this differently