tl;dr This patch is ready to merge but sebb wants to veto it. https://github.com/apache/whimsy/pull/120
The family-first checkbox and family-first form field in secretary workbench are useful for us to detect earlier than we do today a possible conflict in names of ICLA submitters. This handy tool (thanks, sebb) shows that there are many cases where we have accepted ICLAs from people ostensibly with the same names: https://whimsy.apache.org/secretary/icla-dupes We have not had a consistent policy for handling people with the same names, and we should do so. I'd suggest a different thread for that discussion. But not having a policy is not a good reason to not know that we have a conflict. I've commented out the visible part of the tool that initializes LDAP fields from the public name, in case we sometime in future want to provide better values than what we do now. As it is now, we have hundreds of cases where the LDAP name fields are inexplicable. This patch does not fix these cases. Please discuss. Craig > On Aug 7, 2021, at 3:03 PM, GitBox <g...@apache.org> wrote: > > > sebbASF commented on pull request #120: > URL: https://github.com/apache/whimsy/pull/120#issuecomment-894711513 > > > In case it is not obvious, I am -1 on applying this PR without a clear > explanation of the benefit. > > > -- > This is an automated message from the Apache Git Service. > To respond to the message, please log on to GitHub and use the > URL above to go to the specific comment. > > To unsubscribe, e-mail: dev-unsubscr...@whimsical.apache.org > > For queries about this service, please contact Infrastructure at: > us...@infra.apache.org > > Craig L Russell c...@apache.org
