On Tue, Dec 19, 2023 at 4:14 PM Brad Smith bradley.g.smith@gmail.com wrote:
At a minimum, I recommend that the patch include the original values for GOPROXY, GOSUMDB, and GOTOOLCHAIN as comments. This makes it easier to change back to default values. At the moment, one has to visit the relevant web pages.
I lean towards providing upstream defaults in this case with updated content on the developer portal (and elsewhere?) with information on why changing these values would be useful. I agree that the comments in the thread are persuasive (for me).
I agree with this. I'd rather keep the patch than revert to upstream defaults.