[GitHub] [superset] eschutho edited a comment on pull request #19083: chore: update mutator to take kwargs

[GitBox <gi...@apache.org>]

eschutho edited a comment on pull request #19083:
URL: https://github.com/apache/superset/pull/19083#issuecomment-1063298860


   The hope here is that people will use the **kwargs param in their mutator methods instead of hardcoding the params. If they do that, then any future changes will be backward compatible. If they continue to look up the kwargs and add them explicitly to their method, then their implementation will break with future changes to the params that are passed in. 


-- 
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: notifications-unsubscribe@superset.apache.org

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



---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org