petedejoy commented on issue #7670: [AIRFLOW-XXXX] Small grammar fixes in Best 
Practices doc
URL: https://github.com/apache/airflow/pull/7670#issuecomment-596786177
 
 
   My apologies! Jumped the gun on this- was reading through this doc and just 
clicked the `Suggest a change on this page` link, which pushed me to update the 
.rst file directly and make a PR in one fell swoop.
   
   I will say, the doc editing flow for someone who isn't deeply familiar with 
the PR process is a bit odd; I was prompted to make the commit with the default 
message  _before_ being given the PR checklist that tells me what to do with 
commit messages. Definitely on me for not brushing up on the entire process 
beforehand, but for a user reading through docs who just wants to help clean 
them up, it's a bit challenging.
   
   Regardless, I'll roll it back and make sure the proper commit message is 
used. I also have some other updates to this doc that I plan on making.

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


With regards,
Apache Git Services

Reply via email to