yesemsanthoshkumar commented on pull request #279:
URL: https://github.com/apache/airflow-site/pull/279#issuecomment-671075081


   > > 1. Oct 18, 2019
   > >    Same update as Nov 22, 2019. Should I remove one of them?
   > 
   > Yep. mistake. Remove the Nov 22nd one.
   
   Done
   > > 1. April 22, 2016
   > >    Migrating to Apache Phrase points to announcements confluence page. 
Should we repoint this to the website?
   > 
   > Yes please!
   
   Done
   > 
   > Few comments:
   > 
   > 1. It would be great to add some whitespace at the top of the Announcement 
page (as it is for other pages). Currently the header covers half of the first 
line:
   
   Done
   > 1. The list of the backport providers also has far too much space in. 
Maybe better to have them as bullets rather than numbered list to squeeze them 
together.
   
   I didn't notice those spaces between those list items. Removed them now.
   
   > BTW. I am not sure if you realize that but you can easily preview the 
generated pages now. It is enough to download the artifact from the PR, extract 
it and run `python -m http.server` and you will be able to preview it at 
http://localhost:8000
   Thanks for the info. Was checking from the site.sh script preview. Checked 
the above comments with the artifact as well.
   
   Let me know if there are any other issues.


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


Reply via email to