Re: [PR] Expose config to ui via cli support (closes #53645) [airflow]

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on PR #53966:
URL: https://github.com/apache/airflow/pull/53966#issuecomment-3143697845

   > Resolves #53645
   
   Just to comment additionally. We very much prefer that your solution and problem to be solved is described in the commit message - that's why we also often do not need issues to describe problems - PR with description of both the commit and the message is much more valuable. Linking to "issue" is bad because we do not know if 5 years from now we will still have GitHub accounts and those issues. We might decide we move to another hosting - based on Git. So all the details for problems / solutions when we make PR should be part of the commit message - otherwise when you look at the message you see "issue #xxx" and without looking up in Github you have no idea what it is about.
   
   Can you please update the PR description/commit message to contain more information about the problem and solution? Thanks in advance !


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org