pankajastro commented on PR #28850: URL: https://github.com/apache/airflow/pull/28850#issuecomment-1467791465
> > Merging this but will Jarek take the call on CI -- I don't fully like/agree with the CI steps but don't want to block this futher > > Yeah. I completely missed the calling (when I was called 2 weeks ago) - sorry for that. > > @pankajastro - can you please make a follow-up after this one (happy to halp to review and get it done). > > I believe the right way to solve it will be to: > > * add aiobotocore to setup.py in this list (with the comment explaining in both - amazon provider.yaml and in the list, that the two shoudl be synchronized. We > > ``` > # Dependencies needed for development only > devel_only = [ > ``` > > (unless it causes some conflicts, and then we can think what to do) > > * remove the dedicated job to run the tests (and check that in "Providers" section the deferrable tests are executed > > Then it should work out-of-the-box and only when providers are modified. > > @pankajastro - can you make a follow-up PR for that please? Thank you @potiuk for the feedback. I file a PR for this soon. -- 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]
