jscheffl commented on PR #56924:
URL: https://github.com/apache/airflow/pull/56924#issuecomment-3476391657

   You have some problems with static checks in your PR - I assume would be 
helpful to install `prek` and re-format the code. Following this here: 
https://github.com/apache/airflow/blob/main/contributing-docs/03a_contributors_quick_start_beginners.rst
   
   For the errors like "TypeError: DagRunTriggerException.__init__() got an 
unexpected keyword argument 'note'" you need to adjust the parameter for the 
constructor of the Exception class. The exception is in Airflow Core but the 
provider that wants to trigger is in standard provider. Note that the provider 
can be upgraded and used also in other Airflow core versions so if you adjust 
the Exception it will fail in backwards-compatability. I'd leave out the note 
from the Exception, I assume usually the note is not the cause of exception.
   
   


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