potiuk commented on PR #66125: URL: https://github.com/apache/airflow/pull/66125#issuecomment-4348060616
Hmmm. Actually I was thinking about adding tests for the skills .. And I have not decided yet what/whether to do it. This is an interesting approach. However... I think maybe a good idea will be to iterate a bit on it in https://github.com/apache/airflow-steward ? I am **just about** (tomorrow ?) going to move the pr skils there and we can continue iterating there. I also thought (and already created) the https://github.com/apache/airflow-steward-testync. repo for tests fof airlfow-steward- because one of the feature of "airflow-steward" is that you can add it to an external repo as sub-module and use the skils from it ( I already have it in the private security repo of ours - but I want to add it to airflow. This has some nice features - where we have clean split between what is "airflow" and what is "generic" - so I planned to use the "test" repo to test this integration (including automated setup) and re-connecting/re-syncing - and I thought that **maybe** tests could be there - I also wanted to shop around if there are some existing test frameworks for skills - I presume there are quite a number :) -- 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]
