Re: Updating beam container dependencies.

2024-08-09 Thread Danny McCormick via dev
Update - it ended up being a pretty quick fix (just a license issue), so I have an updated PR - https://github.com/apache/beam/pull/32132 Thanks, Danny On Thu, Aug 8, 2024 at 11:12 PM Robert Bradshaw wrote: > No problem, glad to hear this is already the procedure. I'll just do > the minimal cha

Re: Updating beam container dependencies.

2024-08-08 Thread Robert Bradshaw via dev
No problem, glad to hear this is already the procedure. I'll just do the minimal change for now. On Thu, Aug 8, 2024 at 2:11 PM Robert Bradshaw wrote: > > Ah, that's good to know. I'll let you finish this up when you have a chance. > > On Thu, Aug 8, 2024 at 12:11 PM Danny McCormick > wrote: > >

Re: Updating beam container dependencies.

2024-08-08 Thread Robert Bradshaw via dev
Ah, that's good to know. I'll let you finish this up when you have a chance. On Thu, Aug 8, 2024 at 12:11 PM Danny McCormick wrote: > > Thanks for calling this out - we actually have a script to do this > automatically after the release like you're suggesting. This release, it ran > and the res

Re: Updating beam container dependencies.

2024-08-08 Thread Danny McCormick via dev
Thanks for calling this out - we actually have a script to do this automatically after the release like you're suggesting. This release, it ran and the resulting PR is currently assigned to me. I have not