I recently had to resolve a patch conflict while syncing the Microsoft Go fork with upstream because upstream changed src/go.sum and go-crypto-winnative is adding a line in that file.
This PR removes our go.sum file so it does not end up in the patch file. It works because the only entry in our go.sum is a development dependency only used when running go generate ./..., therefore having it in go.sum is nice to have but not required, and in this case is doing more harm than good.
I recently had to resolve a patch conflict while syncing the Microsoft Go fork with upstream because upstream changed
src/go.sum
andgo-crypto-winnative
is adding a line in that file.This PR removes our
go.sum
file so it does not end up in the patch file. It works because the only entry in ourgo.sum
is a development dependency only used when runninggo generate ./...
, therefore having it ingo.sum
is nice to have but not required, and in this case is doing more harm than good.