o-nikolas commented on PR #31001:
URL: https://github.com/apache/airflow/pull/31001#issuecomment-1544852959

   > Yes. Docker-compose and instructions there are part of the docs too :).
   > 
   > And to be brutally honest - I prefer you having hard time now and struggle 
finding all those things out while having the PR red and prevening you from 
merging (and me spending time on helping), rather than having users 3 months 
from now (when released) starting to complain that the things do not work, with 
some poor souls looking on our github issues and triaging them and trying to 
guess why the things are failing from poor description of the users who would 
**just** know they docker-compose fails and their nodes are unhealthy...
   > 
   > The philosophy is:
   > 
   >     * findding/fixing problems as early as we can in the product lifecycle 
(at PR stage is as early as possible)
   > 
   >     * the best way to solve a problem is prevent the problem from 
happening at all
   > 
   > 
   > If we would not follow it, we would not be able to handle the number of 
issues our users would raise :).
   
   Yes, I agree with all of the above, I'm definitely not trying to argue that 
we shouldn't test as much as possible or as early as possible :smile: 
   
   I put up some new changes which should resolve the issue and make the tests 
green! :crossed_fingers: 


-- 
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]

Reply via email to