samredai commented on pull request #1:
URL: https://github.com/apache/iceberg-docs/pull/1#issuecomment-987600519


   > It looks like the Apache 2.0 license header is missing for some README and 
script files, please check and add, thanks!
   > 
   > For Python script, it seems like indentation is off at some places. If we 
do not set up lint in this PR, please make sure we have a task to set it up 
later.
   
   Thanks for catching this! I added the Apache 2.0 license headers everywhere. 
I did a quick auto format of the python scripts with black. I'll follow up this 
PR with a tox file.


-- 
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: dev-unsubscr...@iceberg.apache.org

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


Reply via email to