betodealmeida commented on pull request #11424: URL: https://github.com/apache/incubator-superset/pull/11424#issuecomment-716721447
Looks good to me! Just a heads up, @hughhhh, in the last Superset meetup there was a request to add more context to PRs, specially **why** things are being changed, instead of just **what**. Would be nice to update the description to explain why you're making this change. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
