shollyman

chore: begin migrating away from info.go files (#20020) This PR is the first set of transitions away from handwritten info.go files (used by handwritten clients) towards generating the exported SetGoogleClientInfo functions directly in the client. This allows us to remove the handwritten info.go files. This PR also removes the shell script used to generate new info.go files. related: internal b/489495186

fix(bigquery): improve random ID generation in bigquery (#20111) This moves from math/rand to crypto/rand for the random value producer. It also removes some cruft. Hat tip to jba@ for forwarding this one on. Related: internal b/532634700 --------- Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>

feat: add a basic go module validator tool (#20106) This PR creates a very simple go module validation tool to help us maintain the property of all packages should support some minimum go version. There's a couple of followup items of work here: * wiring this up into a github action so we can see when the invariant is violated * defining an exception mechanism if/when we decide one is warranted I'm deferring these items while I clean up the rest of the violations.

fix(firestore): use higher precision for read times (#20068) The implementation for snapshot reads snapped precision to the nearest second. This change switches the conversion code so that full precision is maintained. Related: internal b/528409928

docs(pubsub): correct MIGRATING.md refs (#20315) Based on PR 14629, corrects links to the new pubsub/v2

chore: wire up mod validation action (#20125) Following up on PR#20106, this change invokes the new validator that runs on all pull requests that touch go.mod files.

chore: only invoke flakybot for nightly (#20457) related: b/555857556

test(firestore): fix another readtime clamping expectation (#20074)

test(compute): skip problematic compute smoke tests (#19989) Skip problematic tests. I tried a simple refactor of the test to limit iteration, but we get the error on the first response. So, skipping these to staunch the bleeding for now. related: internal b/524324270

chore(spanner/benchmarks): cleanup local replace directive (#20316) The use of a local, relative replace directive in the spanner/benchmarks module causes persistent issues with automated dependency updates due to partial propagation when dependencies in the spanner module receive updates. This PR removes out the replace directive, and adds a brief note to README.md describing how to enable local testing. It wasn't clear from the PR that introduced this change that the change meant to be persistent. Fixes: https://github.com/googleapis/google-cloud-go/issues/19926