Repository navigation
fix template_fields in the decorator task.docker - #29586
Conversation
|
Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contribution Guide (https://github.com/apache/airflow/blob/main/CONTRIBUTING.rst)
|
|
Would you be so kind and add a test case covering it ? |
|
will do, Thank you
|
|
Test is added @potiuk |
Fantastic ! Thanks |
Co-authored-by: Andrey Anshin <Andrey.Anshin@taragol.is>
|
And also passing static checks required, for avoid manually fix code styles and other stuff, like use |
… argument type "Sequence[str]"
Co-authored-by: Josh Fell <48934154+josh-fell@users.noreply.github.com>
|
Awesome work, congrats on your first merged pull request! |
|
@jeffery9876 Woohoo! Congrats on your first contribution! 🎉 |
closes: #29585
Add back the template fields that exists under
DockerOperatortotask.docker^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named
{pr_number}.significant.rstor{issue_number}.significant.rst, in newsfragments.