kevinhongzl commented on PR #71592:
URL: https://github.com/apache/airflow/pull/71592#issuecomment-6151838991

   > Hi everyone, I’m not really convinced that #71586 is actually an issue. 
Could you please take a look at my comments here: [#71586 
(comment)](https://github.com/apache/airflow/issues/71586#issuecomment-5677551003)
 I could be wrong, of course, but I don't think any changes are needed here.
   
   Wrapping the template expression within single quotes could work, but the 
rendering result may differ depending on `render_template_as_native_obj` (e.g. 
`'12.0'` vs. `12.0`).  And that's exactly what `render_template_as_native_obj` 
is testing. So +1 for this pr!


-- 
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]

Reply via email to