pion / .goassets

Asset files automatically deployed to Go package repositories
https://pion.ly/
MIT License
9 stars 10 forks source link

Post request to update AUTHORS.txt on forked repo #72

Closed at-wat closed 3 years ago

at-wat commented 3 years ago

In the current contributor list rule, reviewers should ask first time contributors opening PR from forked repository to manually update AUTHORS.txt. It's complicated for the first time contributors and also the reviewers. e.g. https://github.com/pion/srtp/pull/146

This PR adds partial support for forked PRs to generate-authors workflow. If AUTHORS.txt is not updated and the PR is from a forked repo, CI will fail and a patch to update AUTHORS.txt will be posted as a check annotation. example: https://github.com/pion/ci-sandbox/pull/13/files#diff-b897336dda0579c40f3623b68eb6652ded79e206f4cf1af7ceb24b683fd46ff1 image