Re: [PR] Only perform list if delimiter specified [airflow]

2023-12-08 Thread via GitHub
shahar1 commented on PR #36130: URL: https://github.com/apache/airflow/pull/36130#issuecomment-1847709408 Good catch! If it's not too hard, could you try adding tests for both conditions? (e.g., by patching `builtins.list` function) I wouldn't be too strict about it - but if it's poss

Re: [PR] Only perform list if delimiter specified [airflow]

2023-12-08 Thread via GitHub
atrbgithub commented on PR #36130: URL: https://github.com/apache/airflow/pull/36130#issuecomment-1847691555 @eladkal @shahar1 Would you be able to take a look at this if you get a chance? Without this, https://github.com/apache/airflow/pull/34919 should not be released (https://gith

[PR] Only perform list if delimiter specified [airflow]

2023-12-08 Thread via GitHub
atrbgithub opened a new pull request, #36130: URL: https://github.com/apache/airflow/pull/36130 This fixes the issue raised [here](https://github.com/apache/airflow/pull/34919#issuecomment-1846993082) against https://github.com/apache/airflow/pull/34919 When testing locally the list