Uploaded image for project: 'Beam'
  1. Beam
  2. BEAM-14012

Add go fmt CI check

Details

    • Improvement
    • Status: Resolved
    • P2
    • Resolution: Fixed
    • None
    • 2.38.0
    • sdk-go
    • None

    Description

      Right now, we occasionally notice that checked in files have not been formatted properly (e.g. https://github.com/apache/beam/pull/16973). It would be nice to catch these as a precommit check (it could be something as simple as a quick Actions or Jira job that runs:

      ```

      go fmt sdks/go/...
      git diff-index --quiet HEAD – || (echo "Run go fmt before checking in changes" && exit 1)}}

      ```

      That way we push the work to the original authors and let robots enforce linting

      Attachments

        Issue Links

          Activity

            People

              jrmccluskey Jack McCluskey
              damccorm Danny McCormick
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Time Tracking

                  Estimated:
                  Original Estimate - Not Specified
                  Not Specified
                  Remaining:
                  Remaining Estimate - 0h
                  0h
                  Logged:
                  Time Spent - 50m
                  50m