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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to