You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "romsharon98 (via GitHub)" <gi...@apache.org> on 2024/02/07 07:40:51 UTC

[I] Validate templated field assigned also in super constructor [airflow]

romsharon98 opened a new issue, #37216:
URL: https://github.com/apache/airflow/issues/37216

   ### Description
   
   the issue is follow up of this pr: https://github.com/apache/airflow/pull/33786 that add the abilty to checks for the templated field "invalid logic" within the constructor of providers' operators.
   the issue comes from https://github.com/apache/airflow/blob/8961bab6b2ed387bea1193fb3199a1c56ba4455c/scripts/ci/pre_commit/pre_commit_validate_operators_init.py#L176
   and it will add the ability to validate assignment of templated field in super constructor. 
   
   ### Use case/motivation
   
   this issue will close pr's like this https://github.com/apache/airflow/pull/36789 and will prevent similar errors.
   and probably will solve errors from this list https://github.com/apache/airflow/issues/36484
   
   ### Related issues
   
   https://github.com/apache/airflow/issues/36484
   https://github.com/apache/airflow/issues/29069
   
   ### Are you willing to submit a PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Validate templated field assigned also in super constructor [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on issue #37216:
URL: https://github.com/apache/airflow/issues/37216#issuecomment-1931452515

   Thanks for opening your first issue here! Be sure to follow the issue template! If you are willing to raise PR to address this issue please do so, no need to wait for approval.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org