Skip to content

Cloud run: update coverage table server/#4986#887

Merged
ToreMerkely merged 1 commit into
mainfrom
4986-cloud-run-table
May 12, 2026
Merged

Cloud run: update coverage table server/#4986#887
ToreMerkely merged 1 commit into
mainfrom
4986-cloud-run-table

Conversation

@ToreMerkely
Copy link
Copy Markdown
Contributor

No description provided.

@ToreMerkely ToreMerkely enabled auto-merge (squash) May 12, 2026 12:45
@claude
Copy link
Copy Markdown
Contributor

claude Bot commented May 12, 2026

Claude finished @ToreMerkely's task in 1m 29s —— View job


Review of PR #887

  • Run static analysis (build, vet, mod tidy)
  • Read and analyze code changes
  • Post review feedback

Static Analysis

All checks pass: go build, go vet, go mod tidy — no issues.

Summary

Clean documentation-only change to the coverage table in snapshotCloudRunLongDesc. The diff removes the "API" column and trims the "Notes" column to only keep entries that are actionable for users. The table is easier to scan now.

What's good:

  • The simplified table is more user-focused — the removed "API" column was implementation detail that doesn't help CLI users decide what's supported.
  • Keeping the "Use kosli snapshot k8s instead" note on GKE is the right call — it's actionable guidance.
  • The "Not currently supported in the CLI" note on App Engine Standard is a reasonable clarification.

No issues found. This is a straightforward help-text cleanup with no functional changes. LGTM.

@ToreMerkely ToreMerkely merged commit f72464d into main May 12, 2026
20 checks passed
@ToreMerkely ToreMerkely deleted the 4986-cloud-run-table branch May 12, 2026 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants