ericxiao251 commented on PR #65884: URL: https://github.com/apache/airflow/pull/65884#issuecomment-4329993125
> > > > I actually favor #65852 over this alternative here. > > > > > > > > > But this one can be done to v1-2 branch for compatiblity ?.. @ericxiao251 -> maybe you can change the base to `chart/v1-2x-test` if @jscheffl will be ok with it ? > > > > > > Awesome, will wait for his response before changing the base branch :). > > I'd be okay, but we then need to add a small marker/note to the docs of the (new) parameter that this is only transitional for the chart v1.2x and will be fixed generally in chart 2.x Ok I've added the notes, fixed the CI that were failing in `main` and also rebased the PR/commits onto the `chart/v1-2x-test` branch. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
