[GitHub] [airflow] potiuk commented on issue #5808: [AIRFLOW-5205] Check xml files with xmllint + Licenses

2019-08-19 Thread GitBox
potiuk commented on issue #5808:  [AIRFLOW-5205] Check xml files with xmllint + 
Licenses
URL: https://github.com/apache/airflow/pull/5808#issuecomment-522805513
 
 
   This one is really small and mostly about consistency in licence. Rather 
straightforward (the only potential problem is necessity of xmllint  - but this 
is really not a problem because pre-commit will skip that check if you don't 
modify any of the xml files - so you won't need xmllint in most of the cases.


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:
us...@infra.apache.org


With regards,
Apache Git Services


[GitHub] [airflow] potiuk commented on issue #5808: [AIRFLOW-5205] Check xml files with xmllint + Licenses

2019-08-14 Thread GitBox
potiuk commented on issue #5808:  [AIRFLOW-5205] Check xml files with xmllint + 
Licenses
URL: https://github.com/apache/airflow/pull/5808#issuecomment-521523483
 
 
   Made the PR standalone (not depending on series of PRs


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:
us...@infra.apache.org


With regards,
Apache Git Services