sfirke commented on PR #35190:
URL: https://github.com/apache/superset/pull/35190#issuecomment-3312742819

   Thanks @sha174n, this looks great! I might be missing it but it looks like 
you marked one suggestion resolved but didn't implement it. Instead of 
`Regularly apply Superset security updates.`, make it
   ```
   Regularly update to the latest major or minor versions of Superset. Those 
versions receive up-to-date security patches.
   ```
   
   I think it's better to more specifically state the instruction here.
   
   Otherwise this looks good to me!


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to