zclllyybb commented on PR #31409:
URL: https://github.com/apache/doris/pull/31409#issuecomment-1963616679

   basically it's right. but there's still something to be aware of:
   1. please change your PR's title to fit the Doris' style. you could see 
other PR's.
   2. for the added docs, make sure they have proper header like 
   ```md
   ---
   {
       "title": "XXX",
       "language": "en" or "zh-CN"
   }
   ---
   ```
   3. make sure the title in md file(like `# xxx` and "title: XXX" in header) 
is same word with filename. and with the right spelling and the right 
capital/lower cases.
   4. since it's a change not great, you could keep this pr a draft until you 
have deal all the isolated docs. then set it ready for review and then we 
consider to merge it.
   
   thanks for your contribution~


-- 
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-unsubscr...@doris.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org

Reply via email to