[GitHub] [airflow] potiuk edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes
potiuk edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes URL: https://github.com/apache/airflow/pull/6321#issuecomment-543160176 One side-effect of this change is that it will not run tests + pylint in branch pushes (it only runs tests when there is a PR). This however is a nice side effect - I am not porting it to v1-10-test branch but if there will be any other branch we want to push, it will not run the tests and will not block Travis. Only if someone makes PR, the full set of tests will be run. I think this is rather nice behaviour. We will have to adapt it once I port it to v1-10-test (but this will only happen after I port the PROD image there). 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 edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes
potiuk edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes URL: https://github.com/apache/airflow/pull/6321#issuecomment-543160176 One side-effect of this change is that it will not run tests + pylint in branch pushes (it only runs tests when there is a PR). This however is a nice side effect - I am not porting it to v1-10-test branch but if there will be any other branch we want to push, it will not run the tests and will not block Travis. Only if someone makes PR the full set of tests will be run. I think this is rather nice behaviour. We will have to adapt it once I port it to v1-10-test (but this will only happen after I port the PROD image there). 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 edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes
potiuk edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes URL: https://github.com/apache/airflow/pull/6321#issuecomment-543160176 One side-effect of this change is that it will not run tests + pylint in branch pushes (it only runs tests when there is a PR). This hover is a nice side effect - I am not porting it to v1-10-test branch but if there will be any other branch we want to push, it will not run the tests and will not block Travis. Only if someone makes PR the full set of tests will be run. I think this is rather nice behaviour. We will have to adapt it once I port it to v1-10-test (but this will only happen after I port the PROD image there). 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 edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes
potiuk edited a comment on issue #6321: [AIRFLOW-5649] Skips some tests for non relevant changes URL: https://github.com/apache/airflow/pull/6321#issuecomment-543160176 One side-effect of this change is that it will not run tests + pylint in branch pushes (it only run tests when there is a PR). This hover is a nice side effect - I am not porting it to v1-10-test branch but if there will be any other branch we want to push, it will not run the tests and will not block Travis. Only if someone makes PR the full set of tests will be run. I think this is rather nice behaviour. We will have to adapt it once I port it to v1-10-test (but this will only happen after I port the PROD image there). 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