john-bodley commented on a change in pull request #9017: [sip-15] Enabling SIP-15 by default URL: https://github.com/apache/incubator-superset/pull/9017#discussion_r370868110
########## File path: superset/config.py ########## @@ -748,7 +748,7 @@ class CeleryConfig: # pylint: disable=too-few-public-methods # # Note if no end date for the grace period is specified then the grace period is # indefinite. -SIP_15_ENABLED = False +SIP_15_ENABLED = True SIP_15_GRACE_PERIOD_END: Optional[date] = None # exclusive Review comment: @etr2460 I feel like indefinite is preferred to having a hard date, it would be troublesome if someone deployed this change and weren't aware that the grace period had already finished and thus this feature would serve as a hard transition. ---------------------------------------------------------------- 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] With regards, Apache Git Services --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
