coopergillan opened a new pull request #10888:
URL: https://github.com/apache/airflow/pull/10888


   ### What
   
   * Add correct type annotations to ZendeskHook and each method
   * Update one unit test to call an empty dictionary rather than a
   NoneType since the argument should be a dictionary
   
   ### Why
   
   * Building out type annotations is good for the code base
   * The query parameter is accessed with an index at one point, which means 
that it cannot be a `None` type, but should rather be defaulted to an empty 
dictionary if not provided
   
   related: #9708
   
   @mik-laj or others 😅
   
   ---
   **^ Add meaningful description above**
   
   Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)**
 for more information.
   In case of fundamental code change, Airflow Improvement Proposal 
([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals))
 is needed.
   In case of a new dependency, check compliance with the [ASF 3rd Party 
License Policy](https://www.apache.org/legal/resolved.html#category-x).
   In case of backwards incompatible changes please leave a note in 
[UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.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.

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to