New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
scripts: remove github-merge.py #16625
Conversation
c8c1288
to
a83f669
Compare
Concept ACK. Also, I can't verify what script a maintainer runs anyway, can only verify the result. So imo it doesn't matter where this script lives. |
You should link to the new script in the other repository, for transparency. |
This script has been moved to the bitcoin-core/bitcoin-maintainer-tools repository, after discussion in a core dev meeting. The rationale being that it is also useful to other projects, and thus should be moved to a more general repository. Meeting log: http://www.erisian.com.au/bitcoin-core-dev/log-2019-08-15.html See also: bitcoin-core/bitcoin-maintainer-tools#28
a83f669
to
6c27fa0
Compare
ACK 6c27fa0 |
6c27fa0 scripts: remove github-merge.py (fanquake) Pull request description: This script has been moved to the bitcoin-core/bitcoin-maintainer-tools repository, after discussion in a core dev meeting. The rationale being that it is also useful to other projects, and thus should be moved to a more general repository. Meeting log: http://www.erisian.com.au/bitcoin-core-dev/log-2019-08-15.html See also: bitcoin-core/bitcoin-maintainer-tools#28 ~~I'll open a follow up PR to port the documentation to the maintainer tools repo.~~ Ported the docs in bitcoin-core/bitcoin-maintainer-tools#29 ACKs for top commit: laanwj: ACK 6c27fa0 Tree-SHA512: 972070c6893043aebe3b62118d2e42d857553202b1eeb9abca55326f580f25e9859de188a1d0c50b9c7d20bdeeddffc00bcf9ad5a76b60add5c8541956d45489
Needs rebase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
one nit inline. Otherwise looks good.
|
||
### [Verify-Commits](/contrib/verify-commits) ### | ||
Tool to verify that every merge commit was signed by a developer using the above `github-merge.py` script. | ||
Tool to verify that every merge commit was signed by a developer using the `github-merge.py` script. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: s/signed by a developer/signed by a maintainer/
6c27fa0 scripts: remove github-merge.py (fanquake) Pull request description: This script has been moved to the bitcoin-core/bitcoin-maintainer-tools repository, after discussion in a core dev meeting. The rationale being that it is also useful to other projects, and thus should be moved to a more general repository. Meeting log: http://www.erisian.com.au/bitcoin-core-dev/log-2019-08-15.html See also: bitcoin-core/bitcoin-maintainer-tools#28 ~~I'll open a follow up PR to port the documentation to the maintainer tools repo.~~ Ported the docs in bitcoin-core/bitcoin-maintainer-tools#29 ACKs for top commit: laanwj: ACK 6c27fa0 Tree-SHA512: 972070c6893043aebe3b62118d2e42d857553202b1eeb9abca55326f580f25e9859de188a1d0c50b9c7d20bdeeddffc00bcf9ad5a76b60add5c8541956d45489
This script has been moved to the bitcoin-core/bitcoin-maintainer-tools repository,
after discussion in a core dev meeting. The rationale being that it is also useful to
other projects, and thus should be moved to a more general repository.
Meeting log: http://www.erisian.com.au/bitcoin-core-dev/log-2019-08-15.html
See also: bitcoin-core/bitcoin-maintainer-tools#28
I'll open a follow up PR to port the documentation to the maintainer tools repo.Ported the docs in bitcoin-core/bitcoin-maintainer-tools#29