Fokko commented on issue #2708: [AIRFLOW-1746] Add a Nomad operator to trigger job from Airflow URL: https://github.com/apache/incubator-airflow/pull/2708#issuecomment-421754322 @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.
---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected] With regards, Apache Git Services
