potiuk commented on code in PR #24591:
URL: https://github.com/apache/airflow/pull/24591#discussion_r912020838


##########
airflow/cli/__main__.py:
##########
@@ -18,6 +18,7 @@
 # under the License.
 
 from airflow.cli import airflow_cmd
+from airflow.cli.commands import version  # noqa: F401

Review Comment:
   BTW. Just to encourage you @blag - this very much resembles my story in 
Airlfow (which was quoted in 
https://github.com/readme/featured/oss-decision-making). I though I was 100% 
sure I know how the change to our dependency management shoudl be - and I was 
sure it is right by talking to a few people - but when I "announced" publicly 
the change as a "change is coming" and sparked a more wide-spread conversation 
I was reminded that this needs far more of a discussion, delibration and 
thoughts.
   
   I rolled my sleeves up then and  started to do the ground work. Four 
implemented AIPs, becoming a committer, PMC member in the meantime and about 2 
years later we got a process where we had a good solution to the problem I saw 
when I started - very, very different from what I announced, but it was 
handling all the complexities of the problems, I had no idea they existed in 
the first place.
   
   I hope this story might encourage you to follow up.



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