As we merged bitcoin 0.17 (#41) we are now going to merge 0.18 (#727). Merging bitcoin 0.17 happened a bit chaotic: #1013 (squash), #1038 (revert), #1039 (merge).
In general when merging anything the agreement is to squash-merge it using github. For upstream sync merges we do want to keep the upstream history though and perform a merge-merge. Also we manually squash the changes which we do on top of it (did not happen in #1039 but we would like to have it in #727).
Regarding the decision to revert: We had a tie in a quick poll we did, so we went with the default assumptions of never force pushing master.

Checklist
As we merged bitcoin 0.17 (#41) we are now going to merge 0.18 (#727). Merging bitcoin 0.17 happened a bit chaotic: #1013 (squash), #1038 (revert), #1039 (merge).
In general when merging anything the agreement is to squash-merge it using github. For upstream sync merges we do want to keep the upstream history though and perform a merge-merge. Also we manually squash the changes which we do on top of it (did not happen in #1039 but we would like to have it in #727).
Regarding the decision to revert: We had a tie in a quick poll we did, so we went with the default assumptions of never force pushing master.
Checklist