You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I believe the origin of the change is fairly clear with the comment and link to the PR and discussion.
As mentioned, I disagree. There's a reason why Git makes a difference between the author and the committer, and I'm not the author of these commits anymore.
I believe I prematurely consumed your change without first checking your CLA status
I wasn't even aware that signing a CLA is necessary! Neither the README.md nor any CONTRIBUTING.md mentions this. If signing a CLA is required, I urge you to add an according check e.g. via https://github.com/cla-assistant/cla-assistant. Or better yet, use the DCO and https://github.com/probot/dco instead.
Anyway, if I knew a CLA needs to be signed, I wouldn't have contributed. I need to make up my mind what to do now that the contribution already happened.
We will revisit our process for consuming changes to see if we can make any improvements in this area.
Thank you, and yes please, your process needs to be revisited and changed so that authorship in Git history is maintained.
As mentioned, I disagree. There's a reason why Git makes a difference between the author and the committer, and I'm not the author of these commits anymore.
I wasn't even aware that signing a CLA is necessary! Neither the
README.md
nor anyCONTRIBUTING.md
mentions this. If signing a CLA is required, I urge you to add an according check e.g. via https://github.com/cla-assistant/cla-assistant. Or better yet, use the DCO and https://github.com/probot/dco instead.Anyway, if I knew a CLA needs to be signed, I wouldn't have contributed. I need to make up my mind what to do now that the contribution already happened.
Thank you, and yes please, your process needs to be revisited and changed so that authorship in Git history is maintained.
Originally posted by @sschuberth in #6 (comment)
The text was updated successfully, but these errors were encountered: