-
-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
use a Co-authored-by: message in git duet and git as #113
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @gedaiu ! I like how you commented the co-authored by so that it could still be sourced as env vars if desired.
@gedaiu I forgot we require commits to be signed, do you mind signing this one? https://docs.github.com/articles/about-gpg/ |
bbe1fc8
to
ef93e14
Compare
done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks @gedaiu !
git duet-commit -q -m 'Testing set of omega as committer' | ||
run git log -1 --format='%cn <%ce>' | ||
assert_success 'Frances Bar <f.bar@hamster.info.local>' | ||
assert_success 'Zubaz Shirts <z.shirts@pika.info.local>' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This line is failing since git log -1 --format='%cn <%ce>'
only returns the committer. I'm wondering if you meant to add this test at all? I would have expected just to see the ones in git-duet
and git-as
given the change here.
it's a little bit confusing that you don't see all the authors when you run
git duet
andgit as
. This change will make this less confusing :)