You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@beam.apache.org by GitBox <gi...@apache.org> on 2022/11/16 01:13:47 UTC

[GitHub] [beam] lostluck opened a new issue, #24190: [Task][Go SDK]: Update Github action for Go unit testing to use go 1.19 -> blocked on gofmt changes.

lostluck opened a new issue, #24190:
URL: https://github.com/apache/beam/issues/24190

   ### What needs to happen?
   
   Updating .github/workflows/go_tests.yml to use go 1.19 caused the action to fail because we have a `go fmt` check to ensure that all Go code in the repo is using the standard go formatting.
   
   With Go 1.19, doc comments can now have richer formating, which lead to certain ad-hoc formats to become invalid, or change under `gofmt`. See https://tip.golang.org/doc/go1.19#go-doc and https://tip.golang.org/doc/comment for details on this.
   
   While not urgent now, if we update the min SDK version to go 1.19, this becomes a minor blocker. In that event, we should disable the fmt check to allow the action to continue to succeed.
   
   Preferably, we fix the formatting changes prior to that, and run gofmt on the repo and handle the differences, validating the rendering of content with the `godoc` tool.
   
   It is a separate issue to take advantage of the richer documentation features of go1.19.
   
   ### Issue Priority
   
   Priority: 3
   
   ### Issue Component
   
   Component: sdk-go


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [beam] lostluck closed issue #24190: [Task][Go SDK]: Update Github action for Go unit testing to use go 1.19 -> blocked on gofmt changes.

Posted by GitBox <gi...@apache.org>.
lostluck closed issue #24190: [Task][Go SDK]: Update Github action for Go unit testing to use go 1.19 -> blocked on gofmt changes.
URL: https://github.com/apache/beam/issues/24190


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@beam.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org