damccorm commented on PR #17429:
URL: https://github.com/apache/beam/pull/17429#issuecomment-1108585124

   > but how do you feel about merging this after the 2.39 cut, so the first 
release that requires Go 1.18 is 2.40?
   
   That's fine by me - is your goal just to get some additional validation with 
all of us building stuff on Go 1.18 for a while?
   
   > Whenever we merge it though, this PR must also include a note in 
CHANGES.md for whichever version it affects
   
   Yeah, I was planning on adding something in CHANGES.md in a follow up pr 
(along with doc changes) once all the generics work is in - I can also add a 
placeholder in this PR once 2.39 is cut (and the 2.40 release notes are ready) 
though


-- 
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-unsubscr...@beam.apache.org

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

Reply via email to