tirkarthi opened a new pull request, #43512: URL: https://github.com/apache/airflow/pull/43512
This is a follow-up on #43503 . Following are the changes. 1. Made `only_new_ui_files` as an attribute that is true when there are any source files and they contain only new UI changes. Sometimes the PR has no source code changes so instead of diff on 2 empty sets this ensures there are source files and changes in new UI folder with no remaining files left. I am thinking of skipping db tests that now run with "always" that takes 2 mins down from 20 mins earlier but is not needed in new UI only PRs. 2. `docs-build` flag is not used in the CI configuration causing docs build to always run that is not needed in new UI only PRs that don't change any user facing docs. Sometimes the build runs for an hour or more with intermittent failures and retry. This ensures to build docs only when needed. If there are scenarios where the flag is False with docs required then this could be improved. 3. Integration tests are still executed since there is a default list of integrations. Using the `only_new_ui_files` return an empty list so that these tests are skipped for PR with only new UI changes. Each test takes 2 minutes. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
