-
Notifications
You must be signed in to change notification settings - Fork 26
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
Credits for Rob Tovey #97
Comments
Yeah... do we know how to do this? I guess @robtovey committed from an odd account or something... |
So for some reason when I was committing from my laptop it somehow hasn't realised which account to use and has used my laptop name instead of an email address... I think this puts me in the Unrecognized author (no email address) category. Unfortunately there isn't a fix for this, just advice on how to stop it in the future. I don't mind what happens now, I could add a comment line to each of the files I created to make some 'claim' and give a contact address? This should document the contribution in text and (to some extent) by github. |
I think the only option I can see at this point is for you to delete the code you added and commit, then commit adding it back? Which would at least get the lines linked to you, but sounds stupid... |
If you wouldn't mind @robtovey this would be my strong preference. |
ok, I think the best thing to do is try to revert my commit back to the initial branch and then jump to the point at which it was merged. @dnjohnstone, if I go for d83a810 (your big merge) I think the only things I might be unfairly claiming credit for is a couple of formatting edits. Do you mind? Otherwise I could go for the one before ad3dc68? |
Go for it - I'm really not bothered about number of commits :) |
Ok, as far as I can tell things are all in the right place now |
A good portion of our simulation code comes from @robtovey, but isn't ascribed to him by github yet. It would be nice to fix this.
The text was updated successfully, but these errors were encountered: