Closed weiiv closed 5 years ago
@ianco - do you have a good link to instructions on how updating a commit is done? Good to provide that. And how to make it happen by default.
On Feb 21 2019, at 9:27 am, Ian Costanzo notifications@github.com wrote:
Hi @weiiv (https://github.com/weiiv) , we've started to enforce commit signing in this repository, because eventually the code is getting migrated to indy-catalyst and then eventually to a hyperledger repository, so we've started to enforce their requriements. Can you sign your commit? And then we can merge it.
— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub (https://github.com/bcgov/TheOrgBook/pull/761#issuecomment-466089379), or mute the thread (https://github.com/notifications/unsubscribe-auth/AA-IwmNvQxS7jMmHhEQ1P1UQshp5ZpGNks5vPtb0gaJpZM4bH4Ba).
@swcurran I hadn't put together any instructions for this, do we have anything for the indy-catalyst repo from when we enabled this a few weeks ago?
The accepted answer on this is thread should work.
https://stackoverflow.com/questions/13043357/git-sign-off-previous-commits Not my favourite because there is a lot of other stuff on this page, but I would try that first. On Feb 21 2019, at 9:36 am, Ian Costanzo notifications@github.com wrote:
@swcurran (https://github.com/swcurran) I hadn't put together any instructions for this, do we have anything for the indy-catalyst repo from when we enabled this a few weeks ago?
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub (https://github.com/bcgov/TheOrgBook/pull/761#issuecomment-466092777), or mute the thread (https://github.com/notifications/unsubscribe-auth/AA-Iwo168lvDaAhqUqUUgrsPJJSKDPYdks5vPtkXgaJpZM4bH4Ba).
Hi @weiiv , we've started to enforce commit signing in this repository, because eventually the code is getting migrated to indy-catalyst and then eventually to a hyperledger repository, so we've started to enforce their requriements. Can you sign your commit? And then we can merge it.