bugraoz93 commented on PR #65261: URL: https://github.com/apache/airflow/pull/65261#issuecomment-4262363228
> I was just thinking about this. Supporting both positional and flag styles for the same param isn't something most CLI libraries or popular tools do. We're adding maintenance burden, constraining future changes as all CLI libraries might not suppor this feature, and expanding the testing surface forever. Since airflowctl is still pre-1.0, feels like this is the natural window for a clean break rather than committing to maintaining a non-standard approach indefinitely. > > Since the layer we are adding is light, either way works. Just wanted to share my thoughts. > > PS: Sorry for cross posting my message here, since I closed my PR as we have this one open. My PR: #64812 > No need to sorry at all, we are discussing and that is how the best things come out :) Agree, as I mention in your PR as well, that would be the cleanest approach and I also would like to go with it my heart while providing both feature we can ask users and dropping cleaning can also happen incrementally while 1.0 discussions are in. While on the side I was considering even if we are pre 1.0, this would still impact the current user base and cause another migration effort. Let me create a discussion for this. Let's hear from community first. I am also happy to go with clean approach if discussion goes well. Will share the discussion link soon here as well, please join there too :) Until then conveting this to draft -- 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]
