zhongjiajie commented on issue #7729: [AIRFLOW-7066] Use sphinx syntax in 
concepts.rst
URL: https://github.com/apache/airflow/pull/7729#issuecomment-599305130
 
 
   > > BTW, I have some question could not understand, in my PR #7709, raise 
pylint error three time, but `airflow/www/views.py` is in `pylint_todo.txt`, 
which mean will not test by pylint, also test folder will not test by pylint. 
should where the pylint error? (I only change two files)
   > > I test in breeze using `pylint airflow/www/views.py` and `pylint 
test/www/views.py` only `airflow/www/views.py` have some error hint(but it in 
pylint_todo.txt)
   > > @exploy
   > 
   > It is the test/ that is failing. We have two separate pre-commits - pylint 
and pylint-test . The test one is failing in the other build and should be 
fixed there.
   
   Ok, will take a look to the test, BTW, thanks for merging

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


With regards,
Apache Git Services

Reply via email to