potiuk commented on PR #58898: URL: https://github.com/apache/airflow/pull/58898#issuecomment-3599461411
Interesting. Main reason why it was not working was that **recently** - in August - https://github.com/apache/airflow/pull/54258 when I removed some very, very old deprecated commands (`setup-autocomplete` was something that was there initially and we had deprecated command still available till August). > To follow up on this in the future, do you think it's worth it to cover this and similar paths of such setup scripts (and their examples in the docs) with some tests, just in case? I could file an issue to track. Actually this is one of the very few scripts not covered by CI - and I take it almost as a pride point to test all those scripts. This one has not been tested, because it was mostly a helper. But ABSOLUTELY .... Feel free to add it to our CI :) -- 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]
