Skip to content

Format Dataflow wordcount system-test resource with gofmt - #70174

Merged
jason810496 merged 1 commit into
apache:mainfrom
jason810496:fix/google/dataflow-wordcount-gofmt
Jul 22, 2026
Merged

Format Dataflow wordcount system-test resource with gofmt#70174
jason810496 merged 1 commit into
apache:mainfrom
jason810496:fix/google/dataflow-wordcount-gofmt

Conversation

@jason810496

Copy link
Copy Markdown
Member

The Dataflow system-test resource wordcount.go is indented with spaces, so the Go SDK's gofmt prek hook (golines + gofumpt) rewrites it on every repo-wide run (prek run -c go-sdk/.pre-commit-config.yaml gofmt --all-files), leaving contributors with an unrelated dirty file. Tab indentation also matches standard Go formatting and the upstream Apache Beam example this resource copies.

Whitespace-only change: the Dataflow system test compiles the pipeline from source, so behaviour is unchanged.


Was generative AI tooling used to co-author this PR?

The file is indented with spaces, so the Go SDK's gofmt prek hook
(golines + gofumpt) rewrites it on every repo-wide run, leaving
contributors with an unrelated dirty file. Tab indentation also matches
standard Go formatting and the upstream Apache Beam example this
resource copies. Whitespace-only: the Dataflow system test compiles the
pipeline from source, so behaviour is unchanged.
@jason810496
jason810496 requested a review from shahar1 as a code owner July 21, 2026 09:19
@boring-cyborg boring-cyborg Bot added area:providers provider:google Google (including GCP) related issues labels Jul 21, 2026
@jason810496
jason810496 merged commit d2ded22 into apache:main Jul 22, 2026
77 checks passed
@jason810496
jason810496 deleted the fix/google/dataflow-wordcount-gofmt branch July 23, 2026 14:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants