@ndmar The reason is takes soo long to get this merged is the poor quality of 
the PR. Don't get me wrong, but I invested some time in helping, but still a 
lot isn't addressed, for example:

- Incomplete docstring.
- Extremely minimal tests.
- Not following the GIT guidelines regarding the formatting of the commits. 
- Not updating the docs.

I'm happy to do another review, but first please get it to a decent level of 
quality.

[ Full content available at: 
https://github.com/apache/incubator-airflow/pull/2708 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to