nikolauspschuetz commented on PR #69108:
URL: https://github.com/apache/airflow/pull/69108#issuecomment-4937678574
Thanks for the reviews @Vamsi-klu and @SameerMesiah97 🙏
Rebased onto latest \`main\` (the branch had fallen ~200 commits behind) and
force-pushed, which re-triggers the full CI — including the Static checks and
the Compat 2.11.1 provider-distribution leg flagged on the earlier head. Those
looked like stale-base artifacts, so the rebase should clear them; I'll confirm
once the run is green.
Re: `render_template_as_native_obj=True` in the test — it is required here.
`schema_fields` is a list, and native templating is what resolves `{{
var.value.schema_fields }}` back to a Python list rather than its string repr,
which is exactly what the assertion (`operator.schema_fields == SCHEMA_FIELDS`)
verifies; without it the rendered value would be a string and the test would
not meaningfully exercise the fix.
--
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]
