Re: [PATCH v3 0/5] Enable GPG in the Windows part of the CI/PR builds
- From
Jeff King <peff@peff.net>
- Date
- Mar 27, 2020, 09:12 UTC
- Message-ID
- <20200327091206.GB610157@coredump.intra.peff.net>
- In-Reply-To
- <pull.728.v3.git.git.1585236929.gitgitgadget@gmail.com>
On Thu, Mar 26, 2020 at 03:35:23PM +0000, Johannes Schindelin via GitGitGadget wrote:
Show 11 quoted lines
> Changes since v2: > > * Reordered 4/5 before 3/5, as I had intended originally. > > > * Renamed _trace_level to have a trailing underscore, in line with the > surrounding code. > > > * Added a note to the commit message why only lib-gpg.sh loses its > hash-bang line, and no other files in t/.
Thanks, this version looks fine to me. I left a few other comments regarding exit/return in the other part of the thread, but frankly all of it is too arcane and insignificant to spend more brain cycles going back and forth on. So if I convinced/inspired you on that point, feel free to switch it, but otherwise I'm happy with this iteration.
-Peff