Lee-W commented on PR #44172: URL: https://github.com/apache/airflow/pull/44172#issuecomment-2488661042
> > #44026 > > > Can you elaborate what broke in OL? I’m not sure `airflow/dag_processing/collection.py` and `airflow/timetables/assets.py` need to trigger OL tests. Those aren’t really about assets, but more _using_ assets. > > @uranusjr This [issue](https://github.com/apache/airflow/issues/44026) explain more about what broke in OL. Also, this [PR](https://github.com/apache/airflow/pull/44025) was raised to fix OL tests. I asked @Lee-W for the assets file we can remove these files `airflow/dag_processing/collection.py` and `airflow/timetables/assets.py` if needed. I believe @uranusjr's point is more convincing. Although those are related to assets, they are using assets and are unlikely to affect OL. I should have thought it through more thoroughly. -- 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]
