potiuk commented on pull request #9896: URL: https://github.com/apache/airflow/pull/9896#issuecomment-661178544
I like this much better than the checklist one. I agree it becomes very quickly an unnecessary boilerplate. The one problem I have with the "**Here is the meaningful description of my changes.**" the way it is written is that I often provide meaningful description in the commit itself and then it will land above the line. Which I do not really like. So maybe we should adopt the commonly used pattern (and a nice call to action I think as well): ``` --- ^ Add meaningful description above ``` I think this is also nice that there is only one area (above the first ---) that you should edit as PR author. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
