You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/03/09 21:21:45 UTC

[GitHub] [airflow] petedejoy commented on issue #7670: [AIRFLOW-XXXX] Small grammar fixes in Best Practices doc

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:
users@infra.apache.org


With regards,
Apache Git Services