zowe / community

Zowe Community - Sub-projects, Squads, Contribution Guidelines, Meeting Minutes, and more
53 stars 41 forks source link

Are github branch updates removing DCO signoffs? #2092

Open 1000TurquoisePogs opened 1 year ago

1000TurquoisePogs commented 1 year ago

A few people believe they've witnessed that DCO signoffs were fine on a PR prior to clicking the button to update the branch if it is out of date with the target. image

After this point, I personally have seen the green check for DCO signoff turn into a red failure where it says DCO is missing. Unless this is my imagination, it's as if this button can sometimes remove DCO. Or otherwise, the original DCO check was wrong?

Either way, if people can witness this for certain, please note the PR so that we can send this to whoever is the author of whatever tool (the DCO bot, or github itself).

1000TurquoisePogs commented 1 year ago

Adding you John just incase this is due to behavior of github itself since I recall you helped get DCO behavior for us as a sort of pilot/beta program, you might know who to contact if we do spot an issue for sure.

jmertic commented 1 year ago

@1000TurquoisePogs Can you point me to a PR with this happening? Thanks!

1000TurquoisePogs commented 11 months ago

This just happened. No DCO issue, then click the "update branch" button, then DCO issues on every commit. Upon checking the commits, I see that the DCO bot returned green for every commit up until the last one, in which it retroactively thinks the commits are now wrong.

https://github.com/zowe/zen/pull/84/commits

jmertic commented 11 months ago

Thanks @1000TurquoisePogs. I don't see a DCO signoff on any of those commits; are you saying there were DCO signoffs on the commits and they all are gone now?

jmertic commented 11 months ago

Closing the loop - I'd recommend enabling remediation commits for all repos.

https://github.com/dcoapp/app#individual-remediation-commit-support

This will make it easier to clean up DCO signoffs that are missing.

jmertic commented 9 months ago

Seeing if this is still an issue - let me know.