fisjac commented on PR #27773: URL: https://github.com/apache/superset/pull/27773#issuecomment-2045739890
@michael-s-molina could you clarify what you mean. It seems we have a couple options here: 1. build the feature flag and merge in. This would enable building smaller PRs to build out the feature, but possibly creates and unused feature flag if the feature isn't merged. 2. Build the feature as a separate branch and merge in together alongside the feature flag, creating a large PR. We can go either way, but it would seem smaller PRs will be much easier to implement/review. -- 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-unsubscr...@superset.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@superset.apache.org For additional commands, e-mail: notifications-h...@superset.apache.org
